Difference between revisions of "Meetings"

From Minetest Developer Wiki
Jump to navigation Jump to search
m (+brightness)
Line 12: Line 12:
 
'''Remember to post meetings in the [https://github.com/orgs/minetest/teams/engine GitHub org] in advance'''
 
'''Remember to post meetings in the [https://github.com/orgs/minetest/teams/engine GitHub org] in advance'''
  
== 2022-10-09 ==
+
== 2022-10-23 ==
  
 
Add your points here. Most important comes first.
 
Add your points here. Most important comes first.
Line 22: Line 22:
 
'''PR discussion/reviews'''
 
'''PR discussion/reviews'''
  
* [https://github.com/minetest/minetest/pull/11432#issuecomment-1263562675 Add support for attached facedir/4dir nodes (comment)] - Thoughts on adding this as part of the PR? (TurkeyMcMac)
+
* ...
* [https://github.com/minetest/minetest/pull/12739 Add callback on_mapblocks_changed] - Decide whether this is the right approach. We could go for something with node granularity, but it would probably be more expensive. (TurkeyMcMac)
 
* [https://github.com/minetest/minetest/pull/12611 Implement some VoxelManip functionality safely using LuaJIT's FFI library] - Decide whether the added complexity is worth it. (TurkeyMcMac)
 
* [https://github.com/minetest/minetest/pull/12449 Use .md extension for markdown files] - Please decide once and for all (Zughy)
 
* Shader brightness issue
 
  
 
Also consider:
 
Also consider:
Line 32: Line 28:
 
* [https://github.com/minetest/minetest/pulls?q=is%3Aopen+is%3Apr+label%3A%22One+approval%22 "One Approval" PRs] and decide on whether to merge, request changes or close.
 
* [https://github.com/minetest/minetest/pulls?q=is%3Aopen+is%3Apr+label%3A%22One+approval%22 "One Approval" PRs] and decide on whether to merge, request changes or close.
 
* [https://github.com/minetest/minetest/pulls?q=is%3Aopen+is%3Apr+label%3A%22Roadmap%3A+Needs+approval%22 "Roadmap: needs approval" PRs]
 
* [https://github.com/minetest/minetest/pulls?q=is%3Aopen+is%3Apr+label%3A%22Roadmap%3A+Needs+approval%22 "Roadmap: needs approval" PRs]
 +
  
 
= Past Meetings =
 
= Past Meetings =
 +
 +
== 2022-10-09 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/11432#issuecomment-1263562675 Add support for attached facedir/4dir nodes (comment)] - Thoughts on adding this as part of the PR? (TurkeyMcMac)
 +
* [https://github.com/minetest/minetest/pull/12739 Add callback on_mapblocks_changed] - Decide whether this is the right approach. We could go for something with node granularity, but it would probably be more expensive. (TurkeyMcMac)
 +
* [https://github.com/minetest/minetest/pull/12611 Implement some VoxelManip functionality safely using LuaJIT's FFI library] - Decide whether the added complexity is worth it. (TurkeyMcMac)
 +
* [https://github.com/minetest/minetest/pull/12449 Use .md extension for markdown files] - Please decide once and for all (Zughy)
 +
* Shader brightness issue
 +
  
 
== 2022-09-25 ==
 
== 2022-09-25 ==

Revision as of 23:26, 20 October 2022

Core developer meetings are held when needed. They take place in the #minetest-dev IRC channel.

This Wiki page contains the meeting plans for upcoming meetings, and meeting notes for previous meetings.

Only core devs should edit this page. You can raise points to be discussed in an upcoming meeting by contacting a core dev, or messaging on IRC. Please avoid derailing meetings by posting unrelated things whilst discussion about other things is happening.


Upcoming Meetings

Remember to post meetings in the GitHub org in advance

2022-10-23

Add your points here. Most important comes first.

Organization Discussion

  • ...

PR discussion/reviews

  • ...

Also consider:


Past Meetings

2022-10-09

PR discussion/reviews


2022-09-25

PR discussion/reviews


2022-09-11


2022-08-28

Organization Discussion

  • Add some bugs to the 5.6.1 milestone? (Proposal: each core dev pick one and adds it)
  • Two years have passed since the first roadmap brainstorm: create another one to then follow after 5.7? Also to gather feedback

PR discussion/reviews


2022-08-14

PR discussion/reviews


2022-07-31

Organization Discussion

  • Who writes the release changelog?
  • When does the release happen? presumably today

PR discussion/reviews


2022-07-17

PR discussion/reviews

Also consider:

  • "One Approval" PRs and decide on whether to merge, request changes or close.
    • in particular decide whether we want them for 5.6

2022-07-03

Organization Discussion

  • Feature freeze date for 5.6.0
    • Starting on June 17. Close to 6 months release cycle.

PR discussion/reviews

Roadmapped

2022-06-05

Organization Discussion

  • make a 5.5.2 release due to security fixes?
    • yes

PR discussion/reviews

2022-05-22

PR discussion/reviews

2022-05-08

Organization Discussion

  • Gauge interest in 5.5.1 release
    • 3 in favor, 1 neutral
  • Prioritize things for 5.6?
    • planned date: July, PRs will be added to milestone

PR discussion/reviews

  • #11251 (comment) - backwards compat hinders refactors, drop compat for old irrmt revisions this time?
    • <+rubenwardy> anyway. I think the conclusion is that breaking it is fine, with some concerns over ease-of-use during dev

Also consider:

2022-04-24

Organization Discussion

Issues/planning

PR discussion/reviews

Roadmap: needs approval

High-level/Concept approval for:

Other:

  • New spatial index library for entity lookup performance improvements 12040
    • Library needs checking, first PR should also replace existing spatial lib
  • Add bulk ABMs by TurkeyMcMac
    • Unsure of the question, skipped

2022-04-10

Organization Discussion

  • More regular meetings
    • agreed on fortnightly meetings, starting 2022-04-10 and 2022-04-24
  • Review roadmap rules. Remove 1 week limit, use meetings instead
    • raise limit to 1 month, attempt review sooner during regular meetings

PR discussion/reviews

Also have a look at "One Approval" PRs and decide on whether to merge, request changes or close.

MTG discussion/reviews

2021-09-04

Add your points here. Most important comes first.

Organization Discussion

PR discussion/reviews

Also have a look at "One Approval" PRs and decide on whether to merge, request changes or close.

MTG discussion/reviews

2021-01-23

Organization Discussion

  • SDL2 integration into Irrlicht
    • Ensure that adding an "original Irrlicht" compatibility mode is somehow possible if necessary (ex. Android). (mentioned by sfan5)
    • Development on a separate branch to merge when it's usable
    • SDL2 as Irrlicht device. WIP (nerzhul) https://github.com/minetest/minetest/pull/10780
    • SDL2 for text input. WIP (numzero)
  • Plans for 5.4.0 and feature freeze
    • Feature freeze on 30 Jan 2021

PR discussion/reviews

MTG discussion/reviews


2020-12-19

Organization Discussion

  • 5.3.1 or 5.4.0 release date? There have been multiple big security fixes.
    • Perhaps January, depending on how quickly the milestone issues can be resolved
    • Texture alpha warnings issue: sfan5 needs time to work on it
    • game settings fallback issue: needs decision whether to fix overrideDefaults() or the whole Settings structure (https://github.com/minetest/minetest/issues/10555)

PR discussion/reviews

Various concepts to judge:

MTG discussion/reviews


2020-10-03

Logs: http://irc.minetest.net/minetest-dev/2020-10-03#i_5738345

Organization Discussion

  • 5.4.0 release plan -> December? https://github.com/minetest/minetest/milestone/17
    • -> Rough approximated date for release (after 6 months)
  • Roadmap - everyone list 3 goals for development (ie: Improve UI), this can then be condensed into a roadmap of goals.
  • Add trusted contributors as issue triagers. https://github.com/orgs/minetest/teams/engine/discussions/25
    • Suggested users: Wuzzy, Calinou. Wuzzy was already offered core dev, but refused.
    • They would help ensure issue quality (asking for information, editing, closing duplicated) and oragnise issues (labelling, prioritising)
    • This doesn't prevent core developers from doing the above.
    • celeron55 is willing to trial this, paramat is against this.
    • -> Krock, rubenwardy: Demo run with a few chosen people. Rules will be written down during this phase to ensure everything's going well. Needs discussion with paramat.

PR discussion/reviews

MTG discussion/reviews

2020-08-01

Organisation Discussion

  • Theme of 5.4.0 (ie: what to focus on)
  • New core developers
    • New member invited
  • Minimal/Devtest policy: https://github.com/minetest/minetest/issues/9645
    • Wrote comment
  • minetest-mods.github.io needs a library update - could there be any problems? volunteers?
    • postpone, not so relevant

PR discussion/reviews

MTG discussion/reviews


2020-06-06

Organisation Discussion

  • Allow or deny UTF-8 characters in the source code? relevant PR
  • Translations and minetest.conf.example need updating. Volunteers?
    • Reminder. Will be done when feature freeze starts.
  • Feature freeze date
    • Bone rotation issues need to be fixed (or reverted)
    • Freeze in 1 week: 13 Jun (2 weeks for bugfixes after)

PR discussion/reviews

Suspected issues


2020-03-07

Most important first.

Organisation Discussion

PR discussion/reviews

MTG discussion/reviews


2020-01-11

Arranged by: Krock

Most important first.

Organisation Discussion

  • Looking for new Core Developer members
  • What's missing for 5.1.1?

PR discussion/reviews

MTG discussion/reviews


2019-08-10

Arranged by: Krock

PR discussion/reviews


2019-06-29

Arranged by: rubenwardy

Logs: http://irc.minetest.net/minetest-dev/2019-06-29#i_5561978

Organisation discussion

Proposed order of merging formspec PRs

The idea is to merge the most important PR first (top of list) to not drown in conflicts: https://github.com/minetest/minetest/projects/6

2018-July-August

Arranged by: Krock

Organisation discussion

PR discussion/reviews

2018-06-??

Arranged by: Krock

Organisation discussion

PR discussion/reviews

2018-05-19

Arranged by: Krock

Proposed time: 18:00 UTC

Organisation discussion

PR discussion/reviews

2018-05-05

Arranged by: Krock

It's now almost a month since the last meeting - it time for the next one. Proposed time: 18:00 UTC

Organisation discussion

  • When do we have time to release 0.4.17?
    • Still unclear freeze and release dates. 1 MTE needs backporting first
  • Whether to leave the issue number link in a commit message when merging on Github

PR discussion/reviews

2018-04-07

Arranged by: Krock

Organisation discussion

PR discussion/review

MTG discussion/reviews

2018-03-31

Arranged by: Krock

PR discussion/reviews

0.4.17 release schedule

2018-03-10

Arranged by: Krock

Organisation discussion

PR discussion/reviews

0.4.17 release schedule

2017-05-20

 Minetest developer meeting 2017-05-20

Last chance to merge/improve the remaining PR before feature freeze:

Old business:
 * Additional hooks for inventory manipulations callbacks (#4035)
   * Krock was working on it (#5647), but had an issue with a mysterious crash bug.  Progress?
 * Timed move (#1489)
   * Merge/close?
   * nerzhul: close for me
   * nore: Merge
   * Krock: Merge
 * Split block position in SQLite3 database (and use R*Tree) (#1845)
   * Rebased, performance tests to do soon(tm).
   * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock
    deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date)
 * Add list_predict formspec element (#1988)
   * sofar worked on a rebase a while ago.  Close or merge a modern version?
 * New screenshots for website (web#78)  https://github.com/minetest/minetest.github.io/issues/78
   * ShadowNinja: "good" screenshots are fairly subjective and it doesn't look like we're actually
    getting anywhere near a decision in the issue thread, therefore I propose assigning someone to choose and install new screenshots.
   * > Calinou will handle it


New business:
 * First, release blockers:
 * https://github.com/minetest/minetest/pull/5767 Don't add damage flash while punch texture modifier is active #5767
   * > Bugfix, punted to next meeting.
 * https://github.com/minetest/minetest/issues/5782 [CSM] Disable preview mod before release #5782
   * CSM disabled by default anyways since it's experimental
   * > Fixed by #5554
 * https://github.com/minetest/minetest/issues/5728 Block instadig after tool switch #5728
   * > Fixed by #5785
 * nerzhul:
 * https://github.com/minetest/minetest/pull/5732 (CSM nodedefs/itemdefs read)
 * maybe close https://github.com/minetest/minetest/pull/5654
 * Single/Multiplayer tab merge https://github.com/minetest/minetest/pull/5627 (user UI experience)
   * > Merge with minor changes
 * Create world better experience: https://github.com/minetest/minetest/pull/5589
 * Permit to enable/disable mods on CSM  https://github.com/minetest/minetest/pull/5554 (very important)
   * ShadowNina: CSM is experimental, so I don't think any CSM issues should be blockers
     * nerzhul: here the problem if a user enable client side modding his experience will be affected by this preview mod, dedicated to modders)
   * > nerzhul will merge
 * https://github.com/minetest/minetest/pull/5589 (start world after creation)
 * LMDB backend (https://github.com/minetest/minetest/pull/4206)
   * nerzhul: Close? We already have 5 backends to maintain and we already cover all the usecases (client usage, performance usage, reliability usage)


2017-05-13

 Minetest developer meeting 2017-05-13

Old business:
 * Additional hooks for inventory manipulations callbacks (#4035)
   * Krock was working on it (#5647), but had an issue with a mysterious crash bug.
   * > No progress
 * Timed move (#1489)
   * Merge/close?
   * > nerzhul: close for me
   * > nore: Merge
   * > Krock: Merge
 * Split block position in SQLite3 database (and use R*Tree) (#1845)
   * Rebased, performance tests to do soon(tm).
   * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date)
 * Add list_predict formspec element (#1988)
   * sofar worked on a rebase a while ago.  Close or merge a modern version?


New business:
 * New screenshots for website (web#78)  https://github.com/minetest/minetest.github.io/issues/78
   * ShadowNinja: "good" screenshots are fairly subjective and it doesn't look like we're actually getting anywhere near a decision in the issue thread, therefore I propose assigning someone to choose and install new screenshots.
 * Fix 'alpha' property for liquid nodes (#5494)
   * > paramat testing.
 * CSM: Fix undocumented API calls (#5756)



nerzhul: please look at recent PR for this meeting. Feature Freeze is in 8 days
Here is a list to look at:
 * https://github.com/minetest/minetest/pull/5746 (cleanup content_mapblock)
 * https://github.com/minetest/minetest/pull/5732 (CSM nodedefs/itemdefs read)
 * maybe close https://github.com/minetest/minetest/pull/5654
 * Single/Multiplayer tab merge https://github.com/minetest/minetest/pull/5627 (user UI experience)
 * Create world better experience: https://github.com/minetest/minetest/pull/5589
 * Permit to enable/disable mods on CSM  https://github.com/minetest/minetest/pull/5554 (very important)

2017-05-06

Minetest developer meeting 2017-05-06

Old business:
    "Add configurable key bindings" (#1439)
        Rebased by nore, needs tweaks and merge.
    Additional hooks for inventory manipulations callbacks (#4035)
        Krock is working on it, progress?
    Timed move (#1489)
        Merge/close?
    nerzhul: close for me
    Use a settings object for the main settings (#1949)
        Rebased, should be merged soon.
    Split block position in SQLite3 database (and use R*Tree) (#1845)
        Rebased, performance tests to do soon(tm).
        nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part
      (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date)
    Add list_predict formspec element (#1988)
        sofar worked on a rebase a while ago.  Close or merge a modern version?
    Use object property "stepheight" for player stepheight too. (#2056)
        Agreed to merge, but nobody yet assigned.
    Make players respect makes_footstep_sound in the object properties (#2852)
        Agreed to merge, nobody assigned.

New business:
    Add support for multiple listen addresses (#2604)
        Big patch, but can't really be split up much more since everything's interdependent.
    Make the player collisionbox settable (#2738)
    Clean up numeric.h and split FaceFositionCache from it (#3256)
        Up-to-date and looks more-or-less ready to merge, just needs formal approval.
    <nerzhul>:
        https://github.com/minetest/minetest/pull/5361 Background color on textarea/field/pwdfield)
        https://github.com/minetest/minetest/pull/5355: vertical bar in text areas
        Possible close: https://github.com/minetest/minetest/pull/5281 and https://github.com/minetest/minetest/pull/5279
        Documentation update: https://github.com/minetest/minetest/pull/4968 close or not close ?

For future meeting:
    If finished: https://github.com/minetest/minetest/pull/5544 (on_item_use CSM)

2017-04-29

Arranged by: paramat

ShadowNinja is unavailable today, paramat will be hosting in his stead.

Old business: (Things mentioned at previous meetings that haven't yet been resolved)

  • "Add configurable key bindings" (#1439)
    • Rebased by nore, needs tweaks and merge.
  • Additional hooks for inventory manipulations callbacks (#4035)
    • Krock is working on it, progress?
  • Minetest subgame meeting
    • paramat held one at 1900Z
  • Timed move (#1489)
    • Merge/close?
  • Use a settings object for the main settings (#1949)
    • Rebased, should be merged soon.
  • Split block position in SQLite3 database (and use R*Tree) (#1845)
    • Rebased, performance tests to do soon(tm).
    • nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date)
  • Add list_predict formspec element (#1988)
    • sofar worked on a rebase a while ago. Close or merge a modern version?

New business: (mainly oldest or least recently updated PRs that haven't been handled yet)

  • Added formspec element 'KeyEventBox' for charbased input by ninnghazad (#1737)
    • Closed: better done with client-side mods.
  • Use object property "stepheight" for player stepheight too. (#2056)
    • Agreed to merge, but nobody assigned.
  • Add mapgen settings to create world dialog by srifqi (#2561)
    • Merge, but needs changes.

2017-04-22

Minetest developer meeting 2017-04-22

Old business:
    "getTime refactoring" (#1188) not yet merged (nore)
    "Delayed shutdown" merged by nerzhul.
    "Disable Android LevelDB" rebased by nore and merged by nerzhul.
    "Add configurable key bindings" (#1439) rebased by nore, needs tweaks and merge.
    Additional hooks for inventory manipulations callbacks (#4035)
        Needs rebase and cleanup, volunteers?
    Rename .txt to .md (#5451)?

New business:  (PRs obtained by sorting by age on GitHub)
    Schedule Minetest subgame meeting?
    Coverity: https://scan.coverity.com/projects/12441
        Need to triage issues, many legitimate issues to fix.
    Timed move (#1489)
    Added formspec element 'KeyEventBox' for charbased input (#1737)
    Add ability to escape commands with a back-slash (#1768)
        <ShadowNinja> I may want to tweak this so that it doesn't strip the slash when you type "\foo" (or maybe not?
     It should at least work for the client-side dot-commands too)
    Split block position in SQLite3 database (and use R*Tree) (#1845)
    Use a settings object for the main settings (#1949)
        <ShadowNinja> I also unified Initialize and InitiallizeAsync, which was necessary to implement this.
        I think the unification is worth merging even without the settings object change.  This conflicts with just about
        the entire ScriptAPI, so I don't want to rebase it too often :-)
    Add listpredict to the formspec (#1988)
    Add non-global mod namespaces (#2039)



Template

Add your points here. Most important comes first.

Organization Discussion

  • ...

PR discussion/reviews

  • ...

Also consider:

MTG discussion/reviews

  • ...