Difference between revisions of "Meetings"

From Minetest Developer Wiki
Jump to navigation Jump to search
 
(261 intermediate revisions by 7 users not shown)
Line 1: Line 1:
 +
Core developer meetings are held when needed. They take place in the #minetest-dev [https://wiki.minetest.net/IRC IRC] channel.
 +
 +
This Wiki page contains the meeting plans for upcoming meetings, and meeting notes for previous meetings.
 +
 +
'''Only core team 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 =
 
= Upcoming Meetings =
 +
 +
== 2024-11-24 ==
 +
 +
'''Remember to post meetings in the [https://github.com/orgs/minetest/discussions GitHub discussions] in advance'''
 +
 +
Add your points here. Most important comes first.
 +
 +
'''Organization Discussion'''
 +
 +
* foo
 +
 +
'''PR discussion/reviews'''
 +
 +
* bar
 +
 +
Also consider:
 +
 +
* [https://github.com/minetest/minetest/pulls?q=is%3Apr+is%3Aopen+label%3A%22One+approval+%E2%9C%85+%E2%97%BB%EF%B8%8F%22 "One Approval" PRs] and decide on whether to merge, request changes or close.
 +
 +
= Past Meetings =
 +
 +
== 2024-11-10 ==
 +
 +
'''Organization Discussion'''
 +
 +
* release time
 +
** 5.10 is out
 +
 +
'''PR discussion/reviews'''
 +
 +
* How to fix the collision bugs?
 +
** reverted PR #15029 with #15400
 +
 +
== 2024-10-27 ==
 +
 +
'''Organization Discussion'''
 +
 +
* renaming progress
 +
** Halfway through
 +
* feature freeze progress
 +
** RC probably happening tomorrow. Will release in two weeks
 +
 +
== 2024-10-13 ==
 +
 +
Meeting: https://irc.minetest.net/minetest-dev/2024-10-13#i_6208356
 +
 +
'''Organization Discussion'''
 +
 +
* rename time! (Zughy)
 +
** Proceeed to announce on the blog/forums, then rename iteratively
 +
** Blog post: grorp and Zughy are on it
 +
* feature freeze time (Zughy)
 +
** Freeze started. Issues discussed quickly.
 +
 +
== 2024-09-15 ==
 +
 +
'''Organization Discussion'''
 +
 +
* 5.9.1: https://github.com/minetest/minetest/milestone/27
 +
* [https://github.com/minetest/minetest/issues/14543 Add gameid aliases] (herowl)
 +
* (low priority) Future development cycle idea by pgimeno. Almost permanent feature freeze. (again, Krock)
 +
** Idea submission: https://irc.minetest.net/minetest-dev/2024-07-17#i_6185892
 +
** Meeting discussion 1: https://irc.minetest.net/minetest-dev/2024-08-04#i_6189904
 +
** TODO: Set up a poll
 +
* Thoughts on the latest proposed core dev (Zughy)
 +
** Context?
 +
 +
== 2024-09-01 ==
 +
 +
'''Organization Discussion'''
 +
 +
* 5.9.1: https://github.com/minetest/minetest/milestone/27
 +
** Currently 10 PRs ready (merged) that could be included. Not all milestone issues will be fixed in time.
 +
* [https://github.com/minetest/minetest/issues/11568 Separate repo for requesting and discussing new features]. See [https://irc.minetest.net/minetest-dev/2024-09-01#i_6197652 here and next page] for further info (Zughy)
 +
** A few core devs tend to be opposed to this idea. For a definitive result, a poll might be appropriate.
 +
 +
== 2024-08-18 ==
 +
 +
* 5.9.1: https://github.com/minetest/minetest/milestone/27
 +
** Went though one by one
 +
 +
== 2024-08-04 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Finalise 5.9 (Zughy)
 +
** SDL character lookup issue:
 +
** <code>SDL_GetScancodeFromKey(SDLK_SLASH)</code> idea from ''y5nw'' --> https://github.com/minetest/minetest/pull/14894 (hacky workaround, updated)
 +
** See also: meeting discussion
 +
* Future development cycle idea by pgimeno. Almost permanent feature freeze. https://irc.minetest.net/minetest-dev/2024-07-17#i_6185892 (Krock)
 +
** https://irc.minetest.net/minetest-dev/2024-08-04#i_6189904
 +
* https://github.com/minetest/minetest/pull/13987 (observers)
 +
** Krock will check code paths to determine the risks.
 +
 +
 +
== 2024-07-21 ==
 +
 +
'''Organization Discussion'''
 +
 +
* 5.9
 +
 +
'''PR discussion/reviews'''
 +
 +
* https://github.com/minetest/minetest/pull/14749#issuecomment-2222747966 opinions needed
 +
** Krock: "sfan5's comment seems like a good workaround. I could open a PR for that if it's what we'd want"
 +
 +
 +
== 2024-04-28 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Feature freeze when? May 12, release June 1? (Zughy)
 +
** Agreed on the start of June. Also, general consensus about releasing more often
 +
 +
 +
== 2024-03-03 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/13020#issuecomment-1944150506 3d line rendering]: thoughts on grorp's primitive suggestion (Zughy)
 +
** https://irc.minetest.net/minetest-dev/2024-03-03#i_6156739
 +
** general agreement to grorp's comment
 +
** low priority for PR author as of now
 +
** Idea: attach to object bones for more versatility (future PR)
 +
* "One Approval" PR discussion (requested by sfan5)
 +
** 11391: appguru is on it
 +
** 14225 (bulk_get_node): let sfence provide reasons for not closing the PR
 +
** 14319 (nil puncher): answered
 +
** 14347 (pointing range): sfan5 is on it, Desour maybe too
 +
** 14369 (preserve metatables): appguru is on it
 +
* [https://github.com/minetest/minetest/pull/14325 Camera API (draft)]
 +
** Questions & inputs: https://irc.minetest.net/minetest-dev/2024-03-03#i_6156971
 +
 +
== 2024-02-18 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Consider converting unfinished / abandoned PRs to draft rather than closing them. Closing signals "no interest" / rejection, whereas draft signals "not ready" and might be more appropriate for "adoption needed" PRs or similar. (appgurueu)
 +
** Close + "Adoption needed" labels (as always) are better.
 +
* Do we want approvals to be revoked when new commits are pushed? (There's a GH repo setting for this, celeron would have to enable it.) (appgurueu)
 +
** Wiki update WIP
 +
* When do we want 6.0 to happen? After 5.9, after 5.10, even later? (appgurueu)
 +
** Not worth it at the moment, too few breakages on the list (doc/breakages.md) (Krock, sfan5)
 +
** Reference: https://irc.minetest.net/minetest-dev/2024-02-18#i_6153824
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/14154 Simplify bug report form]: stale, we should take a decision (Zughy)
 +
** { Desour, sfan5 } for a simple text template, { rubenwardy, Krock } against
 +
* [https://github.com/minetest/minetest/pull/14225 bulk_get_node] Worth or not? (Krock)
 +
** appgurueu might look into it again to judge the reliability of the performance tests
 +
** node get/set speedup PR: https://github.com/minetest/minetest/pull/14384
 +
 +
== 2024-01-21 ==
 +
 +
'''Organization Discussion'''
 +
 +
* PRs are on the rise and FOSDEM is coming. If it keeps going like this, it'll probably be PR hell all over again. What can we do to avoid both core devs burnouts and contributors being ignored/waiting for ages? Is the roadmap working? (Zughy)
 +
* Suggestion about core devs supporting PRs having to self-assign such PRs, see [https://irc.minetest.net/minetest-dev/2024-01-15#i_6146007 here]. No general "we" should be accepted (Zughy)
 +
** Added the assignment guideline to [https://dev.minetest.net/Git_Guidelines#Issue_and_Pull-Request_Management Git Guidelines]. (Krock)
 +
** Try to assign yourself to PRs that you support to ensure they're not forgotten - even more so on such that are not on the roadmap.
 +
** The guidelines are currently a mess: https://irc.minetest.net/minetest-dev/2024-01-21#i_6147476 - need cleaning up into doc/
 +
 +
'''PR discussion/reviews'''
 +
 +
* What to do about [https://github.com/minetest/minetest/pull/14103 Add drawtype sunken and covered], see my comment (Zughy)
 +
** Left a summary of the discussion (Krock)
 +
* When can we have [https://github.com/minetest/irrlicht/pull/248 Reformat the code irr#248], and irrlicht merge? (DS)
 +
** Only irr#276 would be nice to have merged before import
 +
** Added a milestone for Feb 4 to keep track of the waiting PRs.
 +
* [https://github.com/minetest/irrlicht/pull/281 Irrlicht build fix] - I would be grateful about some opinions. (Krock)
 +
** PR updated with sfan5's suggestion.
 +
* [https://github.com/minetest/minetest/pull/12595 Inventory stack swap fixes] - asking who would have time to review it (Krock)
 +
** Potential reviewers informed.
 +
 +
== 2023-12-10 ==
 +
 +
'''Organization Discussion'''
 +
 +
* 5.8.0 release
 +
** See milestone for open tasks
 +
 +
== 2023-11-12 ==
 +
 +
'''Organization Discussion'''
 +
 +
* 5.8.0 release
 +
** The [https://github.com/minetest/minetest/pull/13872 Formspec Android fix] should be merged beforehand (Krock)
 +
*** grorp also looked into it, without any progress
 +
*** Waiting on progress from any contributor
 +
* Final answer about internal discussion #71 (Zughy)
 +
** Posted a poll at the end of the discussion, waiting for feedback
 +
 +
'''PR discussion/reviews'''
 +
 +
* Can anyone please check the log in [https://github.com/minetest/minetest/issues/13813#issuecomment-1730594420 CJK characters looks been filtered in chat_message while the GUI could display them collectly] so to understand what to do with the issue? (Zughy)
 +
** Krock took care of it, ball in OP's court
 +
* [https://github.com/minetest/minetest/pull/12926 HUD/Formspec replacement PR] status update? (json (IRC) via Krock)
 +
 +
== 2023-10-15 ==
 +
 +
Mainly a revisit of all milestone points without specific PR reviews.
 +
 +
'''Organization Discussion'''
 +
 +
* Feature freeze!
 +
** Revisit all milestone points (Krock)
 +
*** After debundling, only minor or obscure issues are left over. Good enough for a 5.8.0 release (Krock)
 +
** Why did #13885 become a draft? (Krock)
 +
*** grorp is not present for clarifying
 +
 +
== 2023-10-01 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Feature freeze? (Zughy)
 +
** Suggestion (see milestone): keep MTG to have enough time to debundle in 5.9.0 (Krock)
 +
*** Voted for +2 weeks time to get this done.
 +
*** Caveat: rubenwardy and grorp will not be available much in the next few weeks
 +
** Feature freeze on Oct 15, release on Oct 29
 +
** [https://github.com/minetest/minetest/issues/13192 LTO optimization] is not a blocker. Can be moved to 5.9.0 if needed.
 +
* [https://github.com/minetest/minetest/issues/13476 Settings UI tracker] - Plenty of merged PRs, how much is still required for 5.8.0? (Krock)
 +
** Non-blocking issues for an eventual release.
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/11016 Dual wield] - Feedback brainstorming (Krock)
 +
** Responded.
 +
 +
Concept approval for:
 +
 +
* [https://github.com/minetest/minetest/pull/13764 Liquid system] - Concept yes/no? (Krock)
 +
** Concept approved. Waiting for performance test results.
 +
 +
== 2023-09-17 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Feature freeze? (Zughy)
 +
** Milestone set to Oct 15 for feature freeze on Oct 1 (possibly a dev meeting day) (Krock)
 +
** Reserved additional time to assign milestones and to solve the MTG_follow-up issue https://github.com/minetest/minetest/issues/13800
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/13642 Import Irrlicht] (wsor4035)
 +
** Generally agreed but needs addressing Desour's requirements before merge
 +
** It will have to be merged eventually (sfan5)
 +
* [https://github.com/minetest/minetest/pull/13700 Code style guidelines for Java code] There are already guidelines for C++ and Lua code, but not (Android-specific) Java code. It can be as simple as, "Use Android Studio's linter/formatter." (srifqi)
 +
** Volunteers are welcome to extend this page, e.g. srifqi: https://dev.minetest.net/Android_code_style_guidelines
 +
* A follow up for [https://github.com/minetest/minetest/issues/13583 Issue 13583]: Is there a draft already that I missed? This can be great for documenting supported OS/env. (srifqi)
 +
** The suggested document is currently WIP
 +
** Be clear how old toolchains we support, e.g. by supported Ubuntu LTS versions to use newer libraries or C++ revisions (Krock)
 +
* [https://github.com/minetest/minetest/pull/11016 Dual Wielding] has been waiting for a feedback since 6 months. Since we're busy, I'd suggest to prioritise MTG removal and postpone this feature to 5.9, so that core devs can review it without any pressure (Zughy)
 +
** No progress and the issues could yet not be solved. Removed milestone, marked as draft.
 +
 +
Concept approval for:
 +
 +
* [https://github.com/minetest/minetest/pull/13764 Refactored the liquid system, and added an optional feature for liquids to flow to the next slope]
 +
** Comment written to clarify the situation (Krock)
 +
* [https://github.com/minetest/minetest/pull/13789 Fix liquid falling if in "float" group]
 +
** Concept approved by Krock
 +
 +
== 2023-06-25 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Start considering a hypothetical new name, possibly privately. See [https://github.com/minetest/minetest/issues/13510 Here] (Zughy)
 +
** Will be discussed internally
 +
 +
'''PR discussion/reviews'''
 +
 +
* Less than two months to [https://github.com/minetest/minetest/milestone/22 5.8]: we should consider what to do with a few issues/PRs in the milestone to avoid a last month burst (Zughy)
 +
** Not yet urgent. PRs and issues will be resolved when people have time for it.
 +
 +
Also discussed:
 +
 +
* [https://github.com/minetest/minetest/issues/13550 Debundle Minetest Game]
 +
** Question about "featured" marks to make it more visible to newcomers
 +
 +
== 2023-05-28 ==
 +
 +
'''Organization Discussion'''
 +
 +
* welcome srifqi
 +
* 5.7.1 release (justified by [https://irc.minetest.net/minetest-dev/2023-05-23#i_6086412 Android crashes])
 +
 +
'''PR discussion/reviews'''
 +
 +
* Discuss and concept approve 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/pull/12966 make falling node checks check for protection] TL;DR: Some server made ladders falling, and protection doesn't protect from fall through on_punch by other players. Possible solutions include making the registered on_punch/on_dig/on_placenode function overwritable, or only checking for protection in on_punch. What to do? (DS)
 +
*** Removed roadmap approval needed label. (DS)
 +
 +
== 2023-05-14 ==
 +
 +
'''Organization Discussion'''
 +
 +
* close all the Jude's PRs that haven't been adopted (Zughy)
 +
** done
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/11016 "Dual Wielding"] I think OP is waiting for a core dev feedback, see their last comment (Zughy)
 +
** ???
 +
 +
== 2023-04-30 ==
 +
 +
Notes inherited from the meeting 2023-04-16
 +
 +
'''Organization Discussion'''
 +
 +
* now that 5.7 has been released, we should write down a roadmap for 5.8 onwards (Zughy)
 +
** No special action taken. Might need picking up later again.
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/12749 "Move code style guidelines into repo"] Do we want this? (I'm myself neutral now that code blocks are no longer broken on the wiki.) (DS)
 +
** Generally okay but updating it might be cumbersome. Guidelines should apply to all Minetest (group) projects (Krock)
 +
** Closed by author (DS).
 +
* [https://github.com/minetest/minetest/issues/13417 "Conditional textures for nodes, bulk texture changing"] Support for this? Or for the alternative linked in the original post? Or none? (Zughy)
 +
** Yet another feature request. No interest shown. Marked as low priority.
 +
* [https://github.com/minetest/minetest/pull/13464 "Add utility script to update/generate *.tr files"] roadmap approval or rejection?
 +
** Answer posted in https://github.com/minetest/minetest/pull/13437
 +
* [https://github.com/minetest/minetest/pull/13215 Irrlicht as a submodule] - how to fix Android compatibility? Ideas are welcome.
 +
** Fix the Android script later? Waiting on numberZero when they have time (Krock)
 +
 +
== 2023-04-02 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Where should we post next meetings notice? (Zughy)
 +
** rubenwardy moved them to https://github.com/orgs/minetest/discussions
 +
** People with read access: MTE + MTG + triagers
 +
* Complaints about connected glass changes - changes the meaning of glasslike_framed_optional and there's no way for texture packs to control it.
 +
** Reverted for now. Let's keep it as a subject for 5.8.0-dev
 +
* 5.7.0 release
 +
** Aiming for 9 April or earlier. Not enough developer activity in this meeting.
 +
 +
== 2023-03-19 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Current roadmap evaluation and draft of the next one (Zughy)
 +
** Still need focus on UI/UX improvements, entities were not touched at all, dehardcoding shouldn't be a point
 +
 +
== 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'''
 +
 +
* [https://github.com/minetest/minetest/pull/13046 "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)
 +
* [https://github.com/minetest/minetest/pull/12189#issuecomment-1130461922 "Reorder client initialization"] needs sfan's take to unlock the status of the PR (Zughy)
 +
** sfan5 unblocked it; will be reviewed when time comes.
 +
* [https://github.com/minetest/minetest/pull/13266 MariaDB] Yes or No (Krock: Yes) - who could test this easily? (Krock)
 +
** obsolete entry
 +
* [https://github.com/minetest/minetest/pull/11016 "Dual Wielding"] How should we go ahead with this? (sfan5)
 +
** Yes: item definition flag (sfan5 answered)
 +
* [https://github.com/minetest/minetest/pull/12315  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'''
 +
 +
* [https://github.com/minetest/minetest/issues/13072 "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'''
 +
 +
* [https://github.com/minetest/minetest/issues/13037 "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.
 +
* [https://github.com/minetest/minetest/pull/12953 "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)
 +
* [https://github.com/minetest/minetest/pull/12928 "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 [https://github.com/minetest/minetest/issues/12746 new roadmap] (sfan5)
 +
** Commented an analysis of the previous roadmap and suggestions on how to increase "success rate" (Krock)
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/11016 "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'''
 +
 +
* [https://github.com/minetest/minetest/pull/12772 Add vector() constructor] - It seems this proposal is controversial. Its fate should be decided. (TurkeyMcMac)
 +
 +
 +
== 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 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/9863 Set visibility of players/hide them from others] has been rebased twice, it's from 2020. Can anyone have a look at it? Or put it in the 5.7.0 milestone, I don't know (Zughy)
 +
* [https://github.com/minetest/minetest/pull/11465 New physics modifiers] concept discussion (Wuzzy)
 +
* The same goes for [https://github.com/minetest/minetest/pull/9776 Show relative screenshot filename on chat when applicable] (Zughy)
 +
 +
 +
== 2022-09-11 ==
 +
 +
* [https://github.com/minetest/minetest/pull/7712 Item pickup] general concept discussion
 +
* [https://github.com/minetest/minetest/pull/11885 Item movement unit tests] required for bugfix PR [https://github.com/minetest/minetest/pull/12595 #12595]
 +
 +
 +
== 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 [https://github.com/minetest/minetest/issues/10461 first roadmap brainstorm]: create another one to then follow after 5.7? Also to gather feedback
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/12449 Use .md extension for markdown files] - yes or no?
 +
* [https://github.com/minetest/minetest/pull/12145 Doc of set_sun/-moon: Absolute orientation and difference in scale ] - document difference in size or slightly change the scale of both the celestial bodies?
 +
 +
 +
== 2022-08-14 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* https://github.com/minetest/minetest/issues?q=is%3Aopen+is%3Aissue+label%3AQuestion
 +
* [https://github.com/minetest/minetest/pull/12367 Add minetest.get_player_window_information()] - any requirements for merge?
 +
* [https://github.com/minetest/minetest/pull/12480 Settings redesign] - how to resolve issues with focus?
 +
 +
 +
== 2022-07-31 ==
 +
 +
'''Organization Discussion'''
 +
 +
* Who writes the release changelog?
 +
* When does the release happen? presumably today
 +
 +
'''PR discussion/reviews'''
 +
 +
* Work on merging the last PRs left in [https://github.com/minetest/minetest/milestone/19 the milestone]
 +
 +
 +
== 2022-07-17 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* review [https://github.com/minetest/minetest/milestone/19 5.6.0 Milestone]
 +
 +
Also consider:
 +
 +
* [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.
 +
** 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'''
 +
 +
* [https://github.com/minetest/minetest/pull/12367 Add minetest.get_player_window_information()]
 +
** How to resist fingerprinting? -> minor concern, rounding is possible
 +
** Move to player ObjectRef? -> player-specific function. not necessarily.
 +
** Should this be prioritised for 5.6? ->
 +
* [https://github.com/minetest/minetest/pull/12284 Show dep errors in Select Mods modal] - pls someone review
 +
** review shall be done
 +
* [https://github.com/minetest/minetest/pull/12480 Settings redesign] - concept approval, prioritisation, requirements for submission
 +
** "yes but not in 5.6"
 +
* [https://github.com/minetest/minetest/pull/11939 Bouncy improvements] - makes trampolines fun again.
 +
** review shall be done
 +
* [https://github.com/minetest/minetest/issues/12493 repeat_place_time] - what do to about this?
 +
** Krock: write a bounds check or removal PR
 +
 +
Roadmapped
 +
 +
* [https://github.com/minetest/minetest/pull/12483 FOV mouse sensitivity]
 +
* [https://github.com/minetest/minetest/pull/12463 http client fetch]
 +
** not needed
 +
* [https://github.com/minetest/minetest/pull/12353 fix acceleration] / [https://github.com/minetest/minetest/pull/11855 fix gravity calculation]
 +
** approved
 +
* [https://github.com/minetest/minetest/pull/10850 allow server to use media from texture_path]
 +
** flawed, but possibly helpful
 +
* [https://github.com/minetest/minetest/pull/10587 Only count entity as "on ground" if they are colliding on the Y axis]
 +
** Krock + sfan5: rejected.
 +
* [https://github.com/minetest/minetest/pull/9424 userdata command line option]
 +
** OK, but needs changes
 +
* [https://github.com/minetest/minetest/pull/7728 Add keep_newlines argument to wrap_text]
 +
** OK, might need fixing
 +
* [https://github.com/minetest/minetest/pull/7712 Add an item pick up callback (2)]
 +
** seems useful.
 +
 +
== 2022-06-05 ==
 +
 +
'''Organization Discussion'''
 +
 +
* make a 5.5.2 release due to security fixes?
 +
** yes
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/12168 Make minetest.write_json use empty list instead of "null" for empty tables] - is this approach good?
 +
* [https://github.com/minetest/minetest/pull/12355 Raise default windowed resolution to 1280x720 (720p)] - yes or no?
 +
** no
 +
 +
== 2022-05-22 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest/pull/11545 #11545] <- someone should review this --[[User:Sfan5|Sfan5]] ([[User talk:Sfan5|talk]])
 +
** x2048
 +
* [https://github.com/minetest/minetest/pull/12367 #12367] some questions here
 +
** mostly answered before meeting
 +
* https://github.com/minetest/minetest/issues/12353 -- Fix acceleration by appgurueu
 +
 +
== 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'''
 +
 +
* [https://github.com/minetest/minetest/issues/12251#issuecomment-1115477945 #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:
 +
 +
* [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]
 +
* oldest PRs
 +
 +
== 2022-04-24 ==
 +
 +
'''Organization Discussion'''
 +
 +
* What to do with closing out old PRs? Should we start assessing old PRs under the roadmap / assigning people?
 +
** Yes, start marking PRs as supported by core dev / roadmap / needs approval. https://github.com/minetest/minetest/pulls?q=is%3Aopen+is%3Apr+-label%3ARoadmap+-label%3ABugfix+-label%3A%22Supported+by+core+dev%22+sort%3Acreated-asc+
 +
* Release in June?
 +
** Potentially, will discuss 5.6.0 goals in next meeting
 +
 +
'''Issues/planning'''
 +
 +
* See proposal in [https://github.com/minetest/minetest/issues/8601  Optional dependencies being overridden/turned into hard dependencies]
 +
** Accepted
 +
* [https://github.com/minetest/minetest/issues/12092 Calling set_detach() + set_properties() + set_pos() on the same step fails to teleport the player half of the time, especially online]
 +
** Marked as confirmed, to be investigated
 +
 +
'''PR discussion/reviews'''
 +
 +
* Close [https://github.com/minetest/minetest/pull/11854 Shader improvements: Saturation Optimization]?
 +
** Closed.
 +
 +
Roadmap: needs approval
 +
 +
* [https://github.com/minetest/minetest/pull/12181 Single functions to get/set every celestial vault element]
 +
** Approved.
 +
* [https://github.com/minetest/minetest/pull/12118 Make debug.g/setmetatable respect the "__metatable_debug" metatable field]
 +
** Postponed for further inputs.
 +
* [https://github.com/minetest/minetest/pull/12080 WIP: Do not stat /etc/localtime all the time]
 +
** Needs actions or will be closed.
 +
* [https://github.com/minetest/minetest/pull/11946 Add mods list formspec display for /mods cmd]
 +
** Approved but needs a better implementation.
 +
 +
High-level/Concept approval for:
 +
 +
* [https://github.com/minetest/minetest/pull/12208 Add support for translating content title and descriptions]
 +
** Inputs provided.
 +
* [https://github.com/minetest/minetest/pull/12185 Add login/register dialog to separate login/register]
 +
** Inputs provided.
 +
 +
Other:
 +
 +
* New spatial index library for entity lookup performance improvements [https://github.com/minetest/minetest/pull/12040 12040]
 +
** Library needs checking, first PR should also replace existing spatial lib
 +
* [https://github.com/minetest/minetest/issues/12128 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.
 +
 +
* [https://github.com/minetest/minetest/pull/11567 SQLite media cache]
 +
** Closed for lack of benchmarks. Opinion is split on whether sqlite is a good approach
 +
* [https://github.com/minetest/minetest/pull/11843 proller's breaking world]
 +
** Closed due to scale of changes vs prioritisation. Agreement made that a hard network break should not be done
 +
* [https://github.com/minetest/minetest/pull/11630 Add API function minetest.activate_objects_in_area]
 +
** worth more consideration, posted comment in PR
 +
* [https://github.com/minetest/minetest/pull/12131 CSM settings]
 +
** Supported by core dev
 +
* [https://github.com/minetest/minetest/issues/12084 Add text tile modifier]
 +
** Closed as draft and bad approach
 +
* [https://github.com/minetest/minetest/issues/11939 Restore and enhance bouncy behavior by pecksin]
 +
** Supported by core dev
 +
* [https://github.com/minetest/minetest/issues/11016 Dual Wielding]
 +
** concept accepted, no need for an arbitrary amount of wields
 +
* [https://github.com/minetest/minetest/issues/11933 A light_source should not override sunlight]
 +
** Krock brought up an implementation concern
 +
* [https://github.com/minetest/minetest/pull/11073 Add rotation support for wallmounted nodes in 'ceiling' or 'floor' mode]
 +
** concept accepted
 +
 +
'''MTG discussion/reviews'''
 +
 +
* [https://github.com/minetest/minetest_game/pull/2942 Don't block fluid (water, lava) flow by flowers, mushrooms, grass and saplings]
 +
** Does this break builds? Does this count as a feature? '''->''' Closed as too close to a feature
 +
 +
== 2021-09-04 ==
 +
 +
Add your points here. Most important comes first.
 +
 +
'''Organization Discussion'''
 +
 +
'''PR discussion/reviews'''
 +
 +
* Android update for November
 +
** WIP. Waiting for rubenwardy. Not very urgent yet.
 +
* Irrlicht milestone
 +
** Await hecks' OpenGL rewrite until beginning of October.
 +
** .. otherwise remove shadowmap main menu setting and disable within C++
 +
* ItemStack metadata network optimizations - https://github.com/minetest/minetest/pull/11014
 +
** Closed.
 +
* Automatic selection of mods - Adopt or Won't add - https://github.com/minetest/minetest/pull/11274
 +
** Closed.
 +
* Death item duplication and callback fix - https://github.com/minetest/minetest/pull/11445
 +
* Joystick controls - https://github.com/minetest/minetest/pull/11262
 +
* New movement tweak (trivial) - https://github.com/minetest/minetest/pull/11367
 +
** To be discussed later.
 +
* Perhaps provide an alternative fix for https://github.com/minetest/minetest/issues/9725
 +
* TODO reviews
 +
** Dynamic Media V2 https://github.com/minetest/minetest/issues/11550
 +
** Animated particle spanwers https://github.com/minetest/minetest/issues/11545
 +
 +
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'''
 +
 +
* https://github.com/minetest/minetest/issues/9352 get_player_information issue
 +
** No solution yet.
 +
 +
* https://github.com/minetest/minetest/pull/10693 Builtin translate
 +
** Needs re-review. Soon ready for merge prior 5.4.0 freeze
 +
 +
* https://github.com/minetest/minetest/pull/10636 Android, ready for review.
 +
** Gradle error is fixed in master (rebase)
 +
** rubenwardy will review/test
 +
 +
* https://github.com/minetest/minetest/pull/10083 Inventory slots style
 +
** Waiting for merge or another approval (v-rob)
 +
 +
* https://github.com/minetest/minetest/issues/10555 Game settings bug
 +
** Fix Settings for once and all in 5.4.0: https://github.com/minetest/minetest/pull/10819
 +
 +
* https://github.com/minetest/minetest/pull/10265 Formspec image features
 +
** Considered for 5.4.0 prior freeze, but not priority
 +
 +
'''MTG discussion/reviews'''
 +
 +
* https://github.com/minetest/minetest_game/pull/2803 Texture compression issue
 +
** Based on issue https://github.com/minetest/minetest_game/issues/2801
 +
** imagefilters.cpp  imageCleanTransparent()  might need fixing to get white-filled opaque leaves after adding colors to the texture
 +
** Affected textures are whitelisted, but ExeVirus review that again
 +
 +
 +
== 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:
 +
 +
* https://github.com/minetest/minetest/pull/10384 image transparency
 +
* https://github.com/minetest/minetest/pull/10430 mapgen constant
 +
* https://github.com/minetest/minetest/pull/10516 base64 (2)
 +
 +
'''MTG discussion/reviews'''
 +
  
  
 
== 2020-10-03 ==
 
== 2020-10-03 ==
  
−
TODO: Add stuff
+
Logs: http://irc.minetest.net/minetest-dev/2020-10-03#i_5738345
  
 
'''Organization Discussion'''
 
'''Organization Discussion'''
 
* 5.4.0 release plan -> December? https://github.com/minetest/minetest/milestone/17
 
* 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.
 
* Roadmap - everyone list 3 goals for development (ie: Improve UI), this can then be condensed into a roadmap of goals.
 +
** -> https://github.com/minetest/minetest/issues/10461
 
* Add trusted contributors as issue triagers. https://github.com/orgs/minetest/teams/engine/discussions/25
 
* 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.
 
** Suggested users: Wuzzy, Calinou.  Wuzzy was already offered core dev, but refused.
−
** Responsibilities:
+
** They would help ensure issue quality (asking for information, editing, closing duplicated) and oragnise issues (labelling, prioritising)
−
*** Ensuring issue quality - asking for information, editing, closing duplicated.
+
** This doesn't prevent core developers from doing the above.
−
*** Organising - labelling, prioritising.
 
−
*** Core developers can still continue as they are.
 
 
** celeron55 is willing to trial this, paramat is against this.
 
** 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'''
 
'''PR discussion/reviews'''
 
* Decide on a "use_texture_alpha" solution: https://github.com/minetest/minetest/issues/10342
 
* Decide on a "use_texture_alpha" solution: https://github.com/minetest/minetest/issues/10342
−
* Adopt or modify anticheat fix: https://github.com/minetest/minetest/pull/10340
+
** -> sfan5 will take care of it
 +
* <s>Adopt or modify anticheat fix: https://github.com/minetest/minetest/pull/10340</s>
 +
* Require debug priv to view gameplay-relevant debug info (2nd try), add wireframe priv  https://github.com/minetest/minetest/pull/9315
 +
** Requested by GreenXenith
 +
** -> Will be reviewed...
  
 
'''MTG discussion/reviews'''
 
'''MTG discussion/reviews'''
−
 
−
= Past Meetings =
 
  
 
== 2020-08-01 ==
 
== 2020-08-01 ==
Line 157: Line 917:
 
* Unarelith's refactors
 
* Unarelith's refactors
  
−
== July-August 2018 ==
+
== 2018-July-August ==
 
Arranged by: Krock
 
Arranged by: Krock
  
Line 176: Line 936:
 
** Review needed (compat break)
 
** Review needed (compat break)
  
−
== June 2018 ==
+
== 2018-06-?? ==
 
Arranged by: Krock
 
Arranged by: Krock
  
Line 515: Line 1,275:
  
 
</nowiki>
 
</nowiki>
 +
 +
 +
= Template =
 +
 +
Add your points here. Most important comes first.
 +
 +
'''Organization Discussion'''
 +
 +
* ...
 +
 +
'''PR discussion/reviews'''
 +
 +
* ...
 +
 +
Also consider:
 +
 +
* [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]

Latest revision as of 13:59, 13 November 2024

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 team 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

2024-11-24

Remember to post meetings in the GitHub discussions in advance

Add your points here. Most important comes first.

Organization Discussion

  • foo

PR discussion/reviews

  • bar

Also consider:

Past Meetings

2024-11-10

Organization Discussion

  • release time
    • 5.10 is out

PR discussion/reviews

  • How to fix the collision bugs?
    • reverted PR #15029 with #15400

2024-10-27

Organization Discussion

  • renaming progress
    • Halfway through
  • feature freeze progress
    • RC probably happening tomorrow. Will release in two weeks

2024-10-13

Meeting: https://irc.minetest.net/minetest-dev/2024-10-13#i_6208356

Organization Discussion

  • rename time! (Zughy)
    • Proceeed to announce on the blog/forums, then rename iteratively
    • Blog post: grorp and Zughy are on it
  • feature freeze time (Zughy)
    • Freeze started. Issues discussed quickly.

2024-09-15

Organization Discussion

2024-09-01

Organization Discussion

2024-08-18

2024-08-04

Organization Discussion


2024-07-21

Organization Discussion

  • 5.9

PR discussion/reviews


2024-04-28

Organization Discussion

  • Feature freeze when? May 12, release June 1? (Zughy)
    • Agreed on the start of June. Also, general consensus about releasing more often


2024-03-03

PR discussion/reviews

2024-02-18

Organization Discussion

  • Consider converting unfinished / abandoned PRs to draft rather than closing them. Closing signals "no interest" / rejection, whereas draft signals "not ready" and might be more appropriate for "adoption needed" PRs or similar. (appgurueu)
    • Close + "Adoption needed" labels (as always) are better.
  • Do we want approvals to be revoked when new commits are pushed? (There's a GH repo setting for this, celeron would have to enable it.) (appgurueu)
    • Wiki update WIP
  • When do we want 6.0 to happen? After 5.9, after 5.10, even later? (appgurueu)

PR discussion/reviews

2024-01-21

Organization Discussion

  • PRs are on the rise and FOSDEM is coming. If it keeps going like this, it'll probably be PR hell all over again. What can we do to avoid both core devs burnouts and contributors being ignored/waiting for ages? Is the roadmap working? (Zughy)
  • Suggestion about core devs supporting PRs having to self-assign such PRs, see here. No general "we" should be accepted (Zughy)

PR discussion/reviews

  • What to do about Add drawtype sunken and covered, see my comment (Zughy)
    • Left a summary of the discussion (Krock)
  • When can we have Reformat the code irr#248, and irrlicht merge? (DS)
    • Only irr#276 would be nice to have merged before import
    • Added a milestone for Feb 4 to keep track of the waiting PRs.
  • Irrlicht build fix - I would be grateful about some opinions. (Krock)
    • PR updated with sfan5's suggestion.
  • Inventory stack swap fixes - asking who would have time to review it (Krock)
    • Potential reviewers informed.

2023-12-10

Organization Discussion

  • 5.8.0 release
    • See milestone for open tasks

2023-11-12

Organization Discussion

  • 5.8.0 release
    • The Formspec Android fix should be merged beforehand (Krock)
      • grorp also looked into it, without any progress
      • Waiting on progress from any contributor
  • Final answer about internal discussion #71 (Zughy)
    • Posted a poll at the end of the discussion, waiting for feedback

PR discussion/reviews

2023-10-15

Mainly a revisit of all milestone points without specific PR reviews.

Organization Discussion

  • Feature freeze!
    • Revisit all milestone points (Krock)
      • After debundling, only minor or obscure issues are left over. Good enough for a 5.8.0 release (Krock)
    • Why did #13885 become a draft? (Krock)
      • grorp is not present for clarifying

2023-10-01

Organization Discussion

  • Feature freeze? (Zughy)
    • Suggestion (see milestone): keep MTG to have enough time to debundle in 5.9.0 (Krock)
      • Voted for +2 weeks time to get this done.
      • Caveat: rubenwardy and grorp will not be available much in the next few weeks
    • Feature freeze on Oct 15, release on Oct 29
    • LTO optimization is not a blocker. Can be moved to 5.9.0 if needed.
  • Settings UI tracker - Plenty of merged PRs, how much is still required for 5.8.0? (Krock)
    • Non-blocking issues for an eventual release.

PR discussion/reviews

  • Dual wield - Feedback brainstorming (Krock)
    • Responded.

Concept approval for:

  • Liquid system - Concept yes/no? (Krock)
    • Concept approved. Waiting for performance test results.

2023-09-17

Organization Discussion

  • Feature freeze? (Zughy)

PR discussion/reviews

  • Import Irrlicht (wsor4035)
    • Generally agreed but needs addressing Desour's requirements before merge
    • It will have to be merged eventually (sfan5)
  • Code style guidelines for Java code There are already guidelines for C++ and Lua code, but not (Android-specific) Java code. It can be as simple as, "Use Android Studio's linter/formatter." (srifqi)
  • A follow up for Issue 13583: Is there a draft already that I missed? This can be great for documenting supported OS/env. (srifqi)
    • The suggested document is currently WIP
    • Be clear how old toolchains we support, e.g. by supported Ubuntu LTS versions to use newer libraries or C++ revisions (Krock)
  • Dual Wielding has been waiting for a feedback since 6 months. Since we're busy, I'd suggest to prioritise MTG removal and postpone this feature to 5.9, so that core devs can review it without any pressure (Zughy)
    • No progress and the issues could yet not be solved. Removed milestone, marked as draft.

Concept approval for:

2023-06-25

Organization Discussion

  • Start considering a hypothetical new name, possibly privately. See Here (Zughy)
    • Will be discussed internally

PR discussion/reviews

  • Less than two months to 5.8: we should consider what to do with a few issues/PRs in the milestone to avoid a last month burst (Zughy)
    • Not yet urgent. PRs and issues will be resolved when people have time for it.

Also discussed:

2023-05-28

Organization Discussion

PR discussion/reviews

  • Discuss and concept approve or close "Roadmap: needs approval" PRs
    • make falling node checks check for protection TL;DR: Some server made ladders falling, and protection doesn't protect from fall through on_punch by other players. Possible solutions include making the registered on_punch/on_dig/on_placenode function overwritable, or only checking for protection in on_punch. What to do? (DS)
      • Removed roadmap approval needed label. (DS)

2023-05-14

Organization Discussion

  • close all the Jude's PRs that haven't been adopted (Zughy)
    • done

PR discussion/reviews

  • "Dual Wielding" I think OP is waiting for a core dev feedback, see their last comment (Zughy)
    • ???

2023-04-30

Notes inherited from the meeting 2023-04-16

Organization Discussion

  • now that 5.7 has been released, we should write down a roadmap for 5.8 onwards (Zughy)
    • No special action taken. Might need picking up later again.

PR discussion/reviews

2023-04-02

Organization Discussion

  • Where should we post next meetings notice? (Zughy)
  • Complaints about connected glass changes - changes the meaning of glasslike_framed_optional and there's no way for texture packs to control it.
    • Reverted for now. Let's keep it as a subject for 5.8.0-dev
  • 5.7.0 release
    • Aiming for 9 April or earlier. Not enough developer activity in this meeting.

2023-03-19

Organization Discussion

  • Current roadmap evaluation and draft of the next one (Zughy)
    • Still need focus on UI/UX improvements, entities were not touched at all, dehardcoding shouldn't be a point

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:

Retrieved from "http://dev.minetest.net/index.php?title=Meetings&oldid=6675"