Minetest logo

IRC log for #minetest-dev, 2016-03-18

| Channels | #minetest-dev index | Today | | Google Search | Plaintext

All times shown according to UTC.

Time Nick Message
00:25 Miner_48er joined #minetest-dev
01:26 Ritchie joined #minetest-dev
02:15 Anchakor joined #minetest-dev
02:22 paramat ok will soon merge games 758 911 931 932 935 941 945 just need to check for conflicts
02:27 paramat not 945, will do that later
02:29 Void7 joined #minetest-dev
02:29 paramat ok merging the first 6
02:43 kaadmy joined #minetest-dev
02:45 paramat merged =P'
02:55 rubenwardy awesome
02:55 rubenwardy What do I need to do to get game#579 merged?
02:55 ShadowBot https://github.com/minetest/minetest_game/issues/579 -- Add give_initial_stuff API (and a setting) by rubenwardy
03:06 yang2003 joined #minetest-dev
03:07 paramat heh elusive devs
03:07 kaadmy ^ ie you :D
03:07 paramat bug the other devs to review it
03:14 paramat asking directly in-channel is the best way
03:23 sofar game#916 just needs one minor adjustment. Tested OK
03:23 ShadowBot https://github.com/minetest/minetest_game/issues/916 -- Create API for registration of fence gate. by Rui914
03:31 Void7 joined #minetest-dev
03:47 paramat ok
03:56 est31 joined #minetest-dev
03:56 est31 paramat, what's the reason to -1 ? #1768
03:56 ShadowBot https://github.com/minetest/minetest/issues/1768 -- Add ability to escape commands with a back-slash by ShadowNinja
03:57 paramat ok i'll add a reason
04:00 paramat hm actually i don't feel strongly enough to -1, removed
04:00 est31 that's fine too :)
04:04 sofar https://forum.minetest.net/viewtopic.php?f=3&t=14228
04:04 sofar any comments?
04:04 sofar please read
04:14 est31 paramat, i do think we should add them up
04:14 est31 if we don't how should we keep track when a pr is mergeable
04:17 paramat well, is someone disapproves, this doesn't remove the approval of another dev. also, the rules are that a PR is mergeable with approvals from 2 core devs
04:18 paramat but yes discussion and disapprovals have to be addressed
04:19 est31 paramat, so you say that if two devs approve, and two disapprove they can game the system?
04:19 est31 pr gets proposed, gets two +1 and two +1
04:19 est31 gets merged because two +1
04:19 est31 second pr gets proposed, this time by dev who -1ed
04:19 est31 it reverts older pr
04:20 est31 gets two +1 and two -1 again
04:20 est31 and so on and so on
04:22 paramat well i agree that we can't mechanically follow +2 = merge, each situation has to be considered flexibly
04:23 paramat but i think the labels should not be a result of addition, but show that 2 devs approve
04:24 est31 well what are the labels for any way
04:24 paramat sofar good idea
04:24 est31 ^
04:25 paramat perhaps we should use 'controversial' when there are many disapprovals
04:25 est31 paramat, you just added them without a word
04:25 paramat i'm going through PRs adding the labels if it is clear a dev approves
04:26 est31 those labels caused some confusion in the past
04:26 est31 e.g. SN has added a label to one of my PRs
04:26 est31 i wondered what that means
04:27 est31 1. does he think I +1 my own PR, and now he adds the label to track this
04:27 est31 2. does he +1 it himself
04:27 est31 so one could interpret you adding the approval label as approval yourself paramat
04:28 est31 and I do think that controversial changes should have it harder to get merged
04:28 est31 imagine if RBA were a dev paramat
04:29 paramat yes good point about self-approval, i think only the author can self approve. approvals by others are clear from their comment '+1'
04:30 est31 and he +1ed his PR #3496
04:30 ShadowBot https://github.com/minetest/minetest/issues/3496 -- Add 3d_plantlike optional drawtype by RealBadAngel
04:30 est31 i +1 it
04:31 est31 and he +1es it
04:31 est31 so two +1es
04:31 est31 and you and others can do what they want
04:31 paramat i agree
04:32 paramat so the approval labels are just a guide
04:32 est31 IMO it serves as a better guide if you look at the sum
04:37 paramat very useful sun though
04:38 kaadmy what's #3496 do?
04:38 ShadowBot https://github.com/minetest/minetest/issues/3496 -- Add 3d_plantlike optional drawtype by RealBadAngel
04:50 est31 kaadmy, it was closed due to disagreement from coredevs
04:50 est31 (me not included)
04:57 thePalindrome joined #minetest-dev
05:22 est31 paramat, on 2627 the submitter needs a perspective on getting this PR merged
05:22 est31 if we don't +1 PRs and merge them in time, we create the impression (especially to only casual contributors) that we dont merge their stuff
05:23 est31 and that impression is accurate in fact :)
05:23 est31 err
05:23 est31 s/:)/:(/
05:29 paramat yes lots of old half dead PRs, we need to get clean up more
05:29 paramat (- get)
05:31 est31 yeah
05:31 paramat closed it
05:32 paramat i'll go through them again and poke the authors
05:32 est31 paramat, generally closing should be only the last resort
05:32 est31 but in this case I agree, because the PR creator hasn't expressed that they would update the PR
05:33 paramat ok
05:33 paramat generally after a year i will poke for a resolution or a close
05:34 paramat or as per git guidelines, close if no reply after a month
05:34 paramat it's not just our fault, many authors will let stuff rot
05:34 est31 I think the wording is "may be closed"
05:34 est31 not "gets" closed
05:34 est31 but yeah
05:35 est31 well idk
05:35 Warr1024 joined #minetest-dev
05:35 paramat game is looking quite good now
05:35 sofar I'm writing up my wish list for 0.4.15
05:36 paramat 30 each for PRs and issues
05:36 est31 30 open issues?
05:36 est31 idk, we can do it like the nmap guy
05:36 sofar I also suggest we do a massive merge-anything-that-smells-good after 0.4.15 to let it cook
05:36 est31 close each feature request
05:36 paramat yeah 30 open of each
05:37 paramat 0.4.15 -> farmap
05:40 sofar right, that would be one of the things
05:49 sofar https://forum.minetest.net/viewtopic.php?f=7&t=14229
05:49 sofar there, lol
05:49 sofar should keep us busy after 0.4.14
06:06 OldCoder paramat, I've posted an edited version of our mapgen discussion in the other channel
06:06 OldCoder Thank you again for your time
06:06 paramat ok cool
06:10 Hunterz joined #minetest-dev
06:32 est31 <paramat> Answer #3: not possible in core, use a lua voxelmanip to place air below -4000
06:32 ShadowBot https://github.com/minetest/minetest/issues/3 -- Furnace segfault
06:32 est31 placing air is a bad idea
06:32 est31 you mine down then you fall down indefinitely
06:33 est31 and cause countless mapblocks to be generated
06:34 est31 I've seen a better approach: simply teleporting players who exceed map borders
06:35 paramat that was discussing adding a lower realm using a lua mapgen to place air, then lower still another realm
06:35 est31 ah
06:35 est31 okay
06:35 est31 there is still the problem with the lighting, no?
06:36 paramat the conversation is not very clear, with some misunderstandings on both sides
06:37 paramat well i added a 'propagate shadow' bool to both engine and lua voxelmanip, that can stop higher realms shadowing lower ones
06:37 paramat that's enabling floatlands in my next mapgen
06:38 paramat and enables floatlands in mgv7 if we want that
07:19 Megaf joined #minetest-dev
07:21 est31 nic
07:21 est31 e
07:37 Obani joined #minetest-dev
07:45 est31 sofar, seen this? https://github.com/minetest/minetest/issues/3874
07:46 est31 wondering whether its a dupe of  #3873
07:46 ShadowBot https://github.com/minetest/minetest/issues/3873 -- Incorrectly selected node after node change with node highlighting
07:49 Obani oh
07:49 Obani 4aiman still is there ?
08:03 Darcidride joined #minetest-dev
08:42 Amaz joined #minetest-dev
09:05 nrzkt joined #minetest-dev
09:36 nore ~tell est31 not dupe, but probably caused by same code
09:36 ShadowBot nore: O.K.
09:54 DevBox joined #minetest-dev
10:46 Samson1 joined #minetest-dev
10:54 Lunatrius` joined #minetest-dev
10:54 celeron55 joined #minetest-dev
11:04 celeron55 joined #minetest-dev
11:15 celeron55 joined #minetest-dev
11:31 est31 joined #minetest-dev
11:46 celeron55 joined #minetest-dev
11:57 celeron55 joined #minetest-dev
12:07 celeron55 joined #minetest-dev
12:09 Lunatrius` joined #minetest-dev
12:17 Calinou joined #minetest-dev
12:30 celeron55 joined #minetest-dev
12:41 Fixer joined #minetest-dev
12:41 Obani joined #minetest-dev
12:44 Indohmo joined #minetest-dev
13:00 turtleman joined #minetest-dev
13:02 celeron55 joined #minetest-dev
13:03 STHGOM joined #minetest-dev
13:37 celeron55 joined #minetest-dev
13:52 everamzah joined #minetest-dev
14:22 blaze joined #minetest-dev
14:37 kaadmy joined #minetest-dev
14:38 est31 joined #minetest-dev
14:50 est joined #minetest-dev
15:08 davisonio joined #minetest-dev
15:11 hmmmm joined #minetest-dev
15:14 srifqi joined #minetest-dev
15:53 srifqi joined #minetest-dev
16:21 Player_2 joined #minetest-dev
16:24 Hunterz joined #minetest-dev
16:27 Void7 joined #minetest-dev
16:42 SaKeLWorld joined #minetest-dev
16:53 Fixer_ joined #minetest-dev
16:57 Krock joined #minetest-dev
17:03 linkedinyou joined #minetest-dev
17:14 Megaf joined #minetest-dev
17:37 Gael-de-Sailly joined #minetest-dev
18:27 ud1 joined #minetest-dev
18:36 Void7 joined #minetest-dev
19:05 twoelk joined #minetest-dev
19:10 kilbith joined #minetest-dev
19:11 kilbith wouldn't that a good idea to stick this topic up momentarily ? https://forum.minetest.net/viewtopic.php?f=3&amp;t=14228
19:11 kilbith the more help to RBA the better
19:28 Gael-de-Sailly joined #minetest-dev
20:04 celeron55 has that been actually confirmed to be true now?
20:04 celeron55 last time i heard it seemed completely dubious
20:07 blaze joined #minetest-dev
20:09 sofar celeron55: vanessa has talked to rba's girlfiend, we both had conversations with his buddy
20:10 VanessaE yeah, it's unfortunately true :(
20:11 sofar also, we knew he had hearth disease before for a while.
20:11 VanessaE yeah, he was taking meds for extremely high blood pressure.
20:12 celeron55 well that seems kind of unfortunate
20:12 sofar afk a bit for some lunch
20:13 Taoki joined #minetest-dev
20:16 celeron55 as for how this affects the actual subject of this channel, i guess everyone knows our bus factor for graphics related stuff wasn't very high so this might have some actual impact on something other than the feelings of people too
20:18 celeron55 potentially quite a lot
20:21 kilbith but you came back into that stuff so the bus factor is preserved
20:21 celeron55 nope; not even close
20:22 VanessaE meanwhile I've stickied the topic forum topic, per kilbith's suggestion.
20:22 VanessaE s/the topic/the /
20:22 kilbith thanks Vanessa
20:22 VanessaE yw
20:22 Obani Is he alive at least ?
20:22 VanessaE yes
20:23 Obani I mean, will it be possible for him to walk, go into the streets...
20:23 VanessaE no.
20:23 Obani The topic was quite unclear about that :s
20:23 VanessaE he's alive but does not have all of his faculties, to put it mildly.
20:24 VanessaE there was a conversation in #minetest yesterday where basically his best friend said he's been reduced to the mental capacity of a newborn :(
20:24 Obani This is a sad period for Minetest also...
20:24 Obani ow
20:24 Obani ow
20:25 Obani dude
20:25 celeron55 shit happens
20:26 Obani yeah
20:27 celeron55 this just seems very inconvenient as even death could have been less worse
20:27 Obani heads or tails
20:27 kilbith i thought the same too
20:27 Obani What would be the best
20:27 Obani I seriously don't know
20:28 Obani I feel bad to only think about it as a Minetest developer
20:29 celeron55 well, frankly i'm going to go on with my life as if nothing happened, but it must be said that his brain starting to work again would be... more pleasant than not
20:30 Obani And here I'm flying to the forum and see TMT and myholes and all this hapiness
20:30 Obani And close the tab
20:33 celeron55 but i'm sure some people in this community have a closer relationship with RBA and, well, obviously can do whatever they want
20:39 celeron55 there are a few open PRs by RBA hanging around; maybe it would be somewhat tasteful to either merge or close them
20:39 celeron55 dunno
20:39 Fixer #3770 gives very big fps boost for me
20:39 ShadowBot https://github.com/minetest/minetest/issues/3770 -- Fix superflous shader setting updates by ShadowNinja
20:39 kilbith i think a message of support from the leader would give an image of cohesive community
20:39 Fixer but has some lighting bug
20:40 kilbith RBA is a part of the Minetest's identity
20:41 Fixer ofc
20:41 celeron55 well, i can say something, but it will not be exactly what you want to hear
20:42 celeron55 i am not that kind of public speaker
20:44 Obani celeron55, if you mean that you're not a politician then that's all right
20:46 sofar I don't see a need for Celeron55 to say anything official, or anyone, even
20:46 sofar even my message was personal and certainly not official
20:48 Fixer we can only hope for the best
20:49 Obani I think it's a good idea to have an official message
20:49 Obani I mean a good initiative*
20:49 Obani s/a/some
20:53 celeron55 well i am not going to write anything because i never do; if something official is written, then it would be a post in the forum news section by a core developer who is the closest to the matter at hand (not sure who in this case); this is how it's worked for years for everything and it will work now too
20:55 sofar which I already have, so
20:56 celeron55 well; it's in general discussion, but other than that it fits that picture
21:00 celeron55 i replied to that topic so now it is so officially official that nobody can ask for more
21:02 Taoki joined #minetest-dev
21:26 ekem joined #minetest-dev
21:28 Obani you know to talk
21:31 _tutima joined #minetest-dev
21:41 rubenwardy joined #minetest-dev
22:03 Miner_48er joined #minetest-dev
22:07 Obani https://up1.ca/#jtPKo2-NjY6hQaRi_eUFiQ
22:33 davisonio joined #minetest-dev
22:45 Void7 joined #minetest-dev
22:51 turtleman joined #minetest-dev
23:13 Obani joined #minetest-dev
23:13 proller joined #minetest-dev
23:33 kaeza joined #minetest-dev
23:49 proller joined #minetest-dev
23:56 kilbith joined #minetest-dev
23:56 kilbith left #minetest-dev
23:56 turtleman joined #minetest-dev

| Channels | #minetest-dev index | Today | | Google Search | Plaintext