Meetings

From Minetest Developer Wiki
Revision as of 19:55, 5 March 2023 by Krock (talk | contribs) (Move to past meetings)
Jump to navigation Jump to search

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

2023-03-19

Remember to post meetings in the GitHub org in advance

Add your points here. Most important comes first.

Organization Discussion

  • foobar

PR discussion/reviews

  • foobar

Also consider:


Past Meetings

2023-03-05

Organization Discussion

  • Feature freeze? (Zughy)
    • Yes, now (sfan5, x2048).
  • Dedicate the upcoming version to Jude/TurkeyMcMac (Zughy, original proposal by jp)
    • sfan5 will do that when updating the credits
  • We're choking on PRs again: shouldn't MT advertise itself more so to potentially find new core devs and/or recruit existing contributors? (Zughy)
    • There is not much that can be done about it, wait for new devs.

PR discussion/reviews

  • "SSCSM execution" (TurkeyMcMac)
    • Should the PR be reviewed in parts? If so, it can be reviewed at this point.
    • It would be good to divide up the work of implementing the various SSCSM APIs, if other people have time to work on this. See the TODO list in the PR description. To see how to implement SSCSM APIs, you can look at src/script/lua_api/l_csm.cpp and src/script/lua_api/l_csm_nodemeta.cpp.
    • Meeting note: x2048 might have a look at it. It should be merged in small chunks, preferably with unittests (Krock)
  • "Reorder client initialization" needs sfan's take to unlock the status of the PR (Zughy)
    • sfan5 unblocked it; will be reviewed when time comes.
  • MariaDB Yes or No (Krock: Yes) - who could test this easily? (Krock)
    • obsolete entry
  • "Dual Wielding" How should we go ahead with this? (sfan5)
    • Yes: item definition flag (sfan5 answered)
  • Add world-independent storage directory for mods - approved, but there's concerns about how multiple instances of Minetest running will cause issues for modders. Is this acceptable? What should modders do in these situations? Is the design fundamentally flawed (rubenwardy)
    • Ideas: callback-based updates for sane locking (sfan5)


2023-01-08

Organization Discussion

  • "Host the generated Lua API documentation under api.minetest.net" (Zughy)
    • Assigned to rubenwardy to move it to GitHub
  • Feature freeze starting on January 22? I suggest to keep every feature currently listed in the milestone (Zughy)
    • focus on the Privacy Policy (sample available)
    • postponed the main menu (settings) rework to 5.8.0
    • minetest.get_player_window_information reviews requested
    • Dual Wield should be reviewed because it's what the players want

PR discussion/reviews

  • "Static binary builds for Linux on common architectures": yes, no? (Zughy)
    • AppImages are coming and Docker is an universal solution. Additional static binaries are not necessary.
  • "Allow zoom even with overwritten default FOV": There seems to be some disagreement on this in the comments. Is the change acceptable? (TurkeyMcMac)
    • Handled (Krock)
  • "Add node texture variants": What item properties should be allowed to vary by variant? (TurkeyMcMac)
    • Currently supported: "tiles", "overlay_tiles", and "special_tiles"
    • Others that could be supported:
      • "inventory_image" and "wield_image": May be unnecessary as there's a separate PR for settings these using item metadata.
      • "mesh": Could be complicated to implement, I haven't looked into it.
    • Should all properties vary based on the same bitfield ("param2_variant") or is this bad API design? (A single bitfield is not a functional limitation. It is still possible to let properties effectively vary independently by making a variant for each possible combination.)
    • -> Feedbacks given for the concept. Might be discussed again in the next meeting.


2022-12-04

Organization Discussion

  • 5.7 release when? (Zughy)
    • somewhen in January is good (rubenwardy, sfan5)
  • we should decide on the new roadmap (sfan5)
    • Commented an analysis of the previous roadmap and suggestions on how to increase "success rate" (Krock)

PR discussion/reviews

  • "Dual wielding": can someone please take care of this PR if the author doesn't come back? It's a huge feature for modders (Zughy)
    • Author came back. Check the progress as usual.


2022-10-23

PR discussion/reviews


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: