Difference between revisions of "Meetings"

From Minetest Developer Wiki
Jump to navigation Jump to search
(More points)
(Update to current meeting)
Line 1: Line 1:
 
= Upcoming Meetings =
 
= Upcoming Meetings =
  
== 2019-07-X ==
+
== 2019-0X-XX ==
  
 
'''PR discussion/reviews'''
 
'''PR discussion/reviews'''
 
* Custom particle generators for particle spawners https://github.com/minetest/minetest/pull/6688
 
** Should this be closed?
 
* Fix debug.txt growing bigger and bigger https://github.com/minetest/minetest/pull/8382
 
** What to change? Or reject & close?
 
* Punchwear https://github.com/minetest/minetest/pull/6804
 
** Who wants to adopt it? The feature was requested many times.
 
* Add support for set_formspec_prepend in main menu https://github.com/minetest/minetest/pull/8611
 
 
* CSM loading crash fix https://github.com/minetest/minetest/pull/8627
 
* CSM loading crash fix https://github.com/minetest/minetest/pull/8627
 
* Formspec text cursor feature https://github.com/minetest/minetest/pull/8665
 
* Formspec text cursor feature https://github.com/minetest/minetest/pull/8665
* Hide chat when console is open https://github.com/minetest/minetest/pull/8656
 
 
* InvRef: Add add_list https://github.com/minetest/minetest/pull/8466
 
* InvRef: Add add_list https://github.com/minetest/minetest/pull/8466
 
** Wanted feature or useless overhead?
 
** Wanted feature or useless overhead?
 
* Custom item wear display https://github.com/minetest/minetest/pull/8447
 
* Custom item wear display https://github.com/minetest/minetest/pull/8447
 
** Concept discussion. Acceptable PR or is there any better solution?
 
** Concept discussion. Acceptable PR or is there any better solution?
 
  
 
'''MTG discussion/reviews'''
 
'''MTG discussion/reviews'''
 
* Craftguide from pauloue https://github.com/minetest/minetest_game/pull/2396
 
* Craftguide from pauloue https://github.com/minetest/minetest_game/pull/2396
 
* Craftguide from LoneWolfHT https://github.com/minetest/minetest_game/pull/2394
 
* Craftguide from LoneWolfHT https://github.com/minetest/minetest_game/pull/2394
 +
  
 
= Past Meetings =
 
= Past Meetings =
 +
 +
== 2019-08-10 ==
 +
 +
'''PR discussion/reviews'''
 +
 +
* Custom particle generators for particle spawners https://github.com/minetest/minetest/pull/6688
 +
** Should this be closed?
 +
* Fix debug.txt growing bigger and bigger https://github.com/minetest/minetest/pull/8382
 +
** What to change? Or reject & close?
 +
* Punchwear https://github.com/minetest/minetest/pull/6804
 +
** Who wants to adopt it? The feature was requested many times.
 +
* Add support for set_formspec_prepend in main menu https://github.com/minetest/minetest/pull/8611
 +
* Hide chat when console is open https://github.com/minetest/minetest/pull/8656
 +
 +
 
== 2019-06-29 ==
 
== 2019-06-29 ==
  

Revision as of 19:26, 10 August 2019

Upcoming Meetings

2019-0X-XX

PR discussion/reviews

MTG discussion/reviews


Past Meetings

2019-08-10

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

July-August 2018

Arranged by: Krock

Organisation discussion

PR discussion/reviews

June 2018

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)