Time  Nick      Message
06:11 Hijiri    rubenwardy: I agree with making them incompatible
06:11 Hijiri    and also washing away all the compatiblity code for old protocol versions
07:14 nerzhul   i'm pushing master lint fix
07:29 nerzhul   i'm pushing a trivial code modernization on server.cpp (for range loops, nullptr, empty functions)
07:39 nerzhul   i push same operation on ban.cpp/h
12:53 paramat   ~tell Wuzzy client-server compat was already broken by settable player collisionbox so stepheight breakage makes no difference :] but yes i think we need to bump protocol to 36 and make minimum 36
12:53 ShadowBot paramat: O.K.
15:00 jas_      So if you bump protocol to 36 min/max I can't connect to my 0.4.16-dev server with my mt android client from play store? (nrz)
15:03 jas_      (I am assuming this is the case, so I'm going to try building mt on android myself.)
15:05 jas_      Or is that only when strict_protocol_version_check is on.. *wonders how many 0.4.16-dev servers are updating*
15:07 celeron55 dropping *all* previous protocol versions?
15:08 celeron55 https://i.imgur.com/phrl12s.gifv
15:09 jas_      Yes indeed.
15:09 paramat   is your android client on 0.4.16 stable?
15:10 jas_      Lemme check. It's whatever is on play store now.
15:10 paramat   probably stable
15:10 jas_      0.4.16.17
15:11 paramat   oh, that's a dev version?
15:12 paramat   anyway, if you did connect incompatible client and server it would be unplayable anyway: players offset vertically and stepheight broken
15:13 nerzhul   0.4.16.17 is the 0.4.16-stable
15:13 paramat   protocol bump just stops connection before the bugs are experienced
15:13 paramat   ah
15:13 jas_      Not the offset for some reason it looks fine.  Dunno bout the stepheight yet.
15:13 nerzhul   we really should switch to 0.5.0-dev instead of released-dev...
15:13 jas_      I got custon stu models for multiskin
15:14 paramat   your server must be on a commit before 21 july then
15:14 paramat   best keep it that way
15:15 paramat   that was the date of the first big compat breakage
15:15 paramat   nerz i agree
15:16 jas_      nerzhul:
15:17 jas_      Oops sorry.  No i like the dev version and to help test if i can.  Thanks.
15:30 paramat   good that's appreciated
15:30 paramat   you'll just have to use a latest client with it then
15:31 paramat   game#1876 is ready
15:31 ShadowBot https://github.com/minetest/minetest_game/issues/1876 -- Farming: Make cotton look like cotton, add crafted string item by paramat
15:53 jas_      i'm on 940bd93e and the models appear fine with android 0.4.16.17 (using stujones11's modified multiskin.b3d character model on server)
15:53 jas_      dunno about stepheight yet
17:30 Krock     I've been thinking about sending real floats in our network packets instead of using the ugly u32 conversion method. But is the representation of a float in a stream standardized?
17:31 Krock     according to IEEE 754 there are multiple representations possible
18:23 nerzhul   merging #6250 in ~5 mins
18:23 ShadowBot https://github.com/minetest/minetest/issues/6250 -- Modernize client code by nerzhul
18:43 celeron55 <+nerzhul> we really should switch to 0.5.0-dev instead of released-dev...
18:43 celeron55 i think we can consider this
18:44 celeron55 does anyone know why that wouldn't work currently? (technical or community or any reason)
18:45 celeron55 and where is the decision in place to make the next release to be 0.5.0?
18:46 celeron55 ah at least the forum post
18:46 nerzhul   it was a consensus between all devs to say we need to do some breakage and master become 0.5.0
18:46 celeron55 yeah, just checking how it's documented
18:47 nerzhul   i proposed and it was accepted to make a tentative 0.4.17 by cherrypick in stable-0.4
18:47 nerzhul   if i remember i had a PR closed at a point permitting to switch to the new model with -dev associated with real devel version
18:48 celeron55 a point?
18:48 nerzhul   a moment*
18:48 celeron55 was it before 0.4.16?
18:49 nerzhul   yes if i remember
18:49 celeron55 before the tradition hasn't caused that much confusion so there hasn't been reason to change, but now with nothing in the version to give clue about breaking changes it's actually a problem
18:50 nerzhul   i think so, but i see more and more reports of users thinking they are using 0.4.16-dev and it's 0.4.16, not 0.5.0 :(
18:51 celeron55 this is probably going to be a mess for some time, maybe it should be neither 0.4.16 nor 0.5.0
18:52 celeron55 not much wiggleroom between 4 and 5 though 8)
18:52 celeron55 it has to be one or the other
18:53 nerzhul   in many apps, especially python and java you are generally using A.B.C-SNAPSHOT version which means you are working on A.B.C, it also permits to have multiple branches with various snapshot versions, one for release +1, one for patching current release, etc
18:53 paramat   2 compatibility breakages were recently merged so latest seems more 0.5 than 0.4
18:54 celeron55 well, one option is to call it 0.5 as long as breaking changes are being made, and once it's possible to promise no breaking changes anymore, just jump to 0.6 (or 1.0)
18:55 celeron55 then 0.5 can be defined to be "the messy versions in between"
18:56 nerzhul   the version is clear is we switch to a A.B.C-dev instead of (A.B.C - 1)-dev :)
18:56 celeron55 well what i started this discussion with was whether others think that's ok
18:56 celeron55 i don't see a problem
18:59 paramat   eww i'm not keen on jumping to 0.6, it makes more sense for the next major version of MT to be 0.5.x, after all we went from 0.3 to 0.4
19:01 paramat   linking '-dev' to the next version instead of the last seems trivial, unless i'm missing something
19:03 sofar     I could care less about numbers. I'm perfectly fine with some projects that just label their versions '53', '54', etc
19:03 sofar     even if they release monthly or more
19:06 celeron55 i think nerzhul is free to change the version to 0.5.0-dev then
19:16 KaadmY    sofar: It's "couldn't care less" :((((
19:17 KaadmY    It's an idiom but it's the opposite of the meaning
19:19 sofar     I could care less :P
19:23 KaadmY    Good
19:47 celeron55 americans like their idioms with opposite meanings
19:49 celeron55 but i don't think that's one of them
19:50 celeron55 maybe it is, i never know
19:58 lisac     hey celeron55, I added Minetest to Gamebanana, do you think that is a good idea?
19:58 lisac     also, maybe you or some core devs could apply for moderator position there.
19:58 lisac     http://gamebanana.com/games/6020
20:06 Fixer     Minetest 416
20:07 Fixer     more than less
20:09 Fixer     retracting, less is at least version 480 now
20:13 paramat   game#1877
20:13 ShadowBot https://github.com/minetest/minetest_game/issues/1877 -- Minimap: Add map mod with craftable map by paramat
20:24 VanessaE  "If a player loses the map item then the minimap stays usable for the rest of the game session but of course is disabled on next game session"
20:25 VanessaE  you're going to want to fix that somehow,.
20:25 VanessaE  Wuzzy will complain :P
20:31 paramat   nah, just not worth a regular checking function, it can be considered player memory of the map :]
20:42 paramat   but it's possible
20:42 Fixer     "If a player loses the map item then the minimap stays usable for the rest of the game sessio" thats no go
20:42 Fixer     thats just silly
20:42 Fixer     silly over 9000
20:46 sofar     it wouldn't be too hard to add the needed callbacks to handle dropping of the map or losing it
20:46 sofar     I would even go as far as making the minimap only show when the map item is in the hotbar, making it even more easy to track removal
20:47 VanessaE  players will object to that
20:47 VanessaE  (given the size of the hotbar)
20:48 sofar     right, but it would be close to what MC does
20:48 VanessaE  if it were up to me, the map would be in its own slot, can't be dropped unless its moved to the main inventory first
20:48 sofar     also possible
20:48 VanessaE  that would probably be the easiest way
20:48 VanessaE  though unified inventory will need an update
20:48 VanessaE  would need*
20:50 paramat   i might be able to make the check insist on hotbar, good idea
20:50 VanessaE  make sure it works with dreambuilder's hotbar size
20:50 VanessaE  (max 23 slots, default 16)
20:51 paramat   anyway, this is WIP first draft, we can discuss a regular checking function, i expect it will be insisted on
20:52 paramat   it already makes no sense that a map enables underground x-ray vision, but later i intend to split minimap radar mode into a separate HUD flag
20:54 VanessaE  isn't the radar mode still broken anyway?
20:55 paramat   and of course, this makes no sense that it doesn't record all areas visited, maybe that will become possible later
20:55 paramat   erm i don't know of a breakage
20:55 VanessaE  I don't remember the details
20:56 VanessaE  but a couple of people said that what's shown in the radar mode isn't correct
20:57 Raven262  It does behave odd.
20:57 paramat   if so please detail in an issue
20:57 Raven262  Like it is refreshed only after one has passed a specific height.
20:58 Raven262  And sometimes, if you follow that map, you'll get into nothing while you were trying to reach a cave that was displayed :P
20:58 sofar     you may have to go up or down, it's confusing
20:59 Raven262  I'm curious, how hight above or below the player does it scan?
20:59 Shara     Guessing which way is part of the fun
21:02 paramat   ok, i'll add a regular map checking function, it'll bypass the criticism :]
21:02 paramat   yeah, obviously what is shown may not be on your level
21:02 paramat   that's not a bug
21:05 twoelk    indeed the radar only shows the percentage of air in a region
21:18 paramat   if the scanned height was less it would show less, so ..
21:24 paramat   btw game#1876 is ready
21:24 ShadowBot https://github.com/minetest/minetest_game/issues/1876 -- Farming: Make cotton look like cotton, add crafted string item by paramat
21:24 twoelk    I think the radar map is quit useful as it is. It's perfect for easily aligning tunnels and there are enough ingame tricks to use it's hints without cheating