Opinons needed: Possible change in map block serialization
- paramat
- Developer
- Posts: 3700
- Joined: Sun Oct 28, 2012 00:05
- GitHub: paramat
- IRC: paramat
- Location: UK
Opinons needed: Possible change in map block serialization
This PR fixes occasional light bugs caused by the cooling of lava https://github.com/minetest/minetest/pull/4682 but comes at a cost:
"To introduce the new flags I had to increase the map block serialization version, this is why you can not play on your maps on other versions of Minetest after you opened them in this version.
Of course old clients still can play, just the server has to be new to be able to read the map."
If this PR is merged, once you open a world with a server you cannot then afterwards open that world with an older server. This includes singleplayer as that is a server and client running on one machine.
"To introduce the new flags I had to increase the map block serialization version, this is why you can not play on your maps on other versions of Minetest after you opened them in this version.
Of course old clients still can play, just the server has to be new to be able to read the map."
If this PR is merged, once you open a world with a server you cannot then afterwards open that world with an older server. This includes singleplayer as that is a server and client running on one machine.
Last edited by paramat on Sun Feb 12, 2017 23:02, edited 2 times in total.
- addi
- Member
- Posts: 666
- Joined: Thu Sep 20, 2012 03:16
- GitHub: adrido
- Location: Black-Forest, Germany
Re: Opinons needed: Possible change in map block serializati
If this does not introduce any other bugs, I would fine with it.
Re: Opinons needed: Possible change in map block serializati
Let me answer with a song
https://www.youtube.com/watch?v=HWPJ8Gvh2C4
I think, progress is more important than backwards compatibility at any cost
https://www.youtube.com/watch?v=HWPJ8Gvh2C4
I think, progress is more important than backwards compatibility at any cost
-
- Member
- Posts: 115
- Joined: Sun Sep 06, 2015 12:23
- GitHub: Thomas--S
- IRC: Thomas-S
- In-game: Thomas-S
- Location: Bavaria, Germany
Re: Opinons needed: Possible change in map block serializati
I think that less bugs are more important than backwards compatibility.
My mods: ts_furniture | ts_doors
-
- Member
- Posts: 73
- Joined: Wed Jan 27, 2016 09:43
Re: Opinons needed: Possible change in map block serializati
From a position of complete ignorance on what/why/where ''map block serialization'' is here, just general-principle questions would be :paramat wrote:. . . you can not play on your maps on other versions of Minetest after you opened them in this version . . . the server has to be new to be able to read the map . . . If this PR is merged, once you open a world with a server you cannot then afterwards open that world with an older server . . .
Does ''server'' include the singleplayer use-case too..?
Are there any feasible longer-term gotchas, unintended consequences, just this side of ''unforeseen'' outcomes..?
Will there be big clear pre-world-launch warnings about irreversibility, like I've seen for various other software over the years..?
Does the change affect the whole (huge) world-database, and if not, could any sort of automatic file-backup mitigate the risk of no-path-back breakage for users wanting to test it..?
Or again, if the huge database is not directly affected, then could there be potentially parallel ways to access the same data, as per the extensions to ISO9660 and hybrid filesystem views of data on optical discs..?
.
Re: Opinons needed: Possible change in map block serializati
I think, the server admin is responsible by himself for backups or to read the doc BEFORE he upgrades anything.CuriousNoob wrote:[
Does the change affect the whole (huge) world-database, and if not, could any sort of automatic file-backup mitigate the risk of no-path-back breakage for users wanting to test it..?
RTFM first
- paramat
- Developer
- Posts: 3700
- Joined: Sun Oct 28, 2012 00:05
- GitHub: paramat
- IRC: paramat
- Location: UK
Re: Opinons needed: Possible change in map block serializati
Discussing on IRC dev channel the devs are thinking this is worth it, the 1st PR will fix more lighting bugs than just large areas of lavacooling, and helps make the 2nd PR more effective.
The first PR (which increases mapblock serialisation version) could be merged as soon as a few days, maybe a week, so this is a warning, and of course if it goes ahead it will be announced here and probably in a dedicated thread in this news subforum.
This does affect singleplayer yes because singleplayer is actually a server and a client running on a single machine.
I'm not knowledgeable about the more technical questions about the world database, maybe others will reply here.
The first PR (which increases mapblock serialisation version) could be merged as soon as a few days, maybe a week, so this is a warning, and of course if it goes ahead it will be announced here and probably in a dedicated thread in this news subforum.
This does affect singleplayer yes because singleplayer is actually a server and a client running on a single machine.
I'm not knowledgeable about the more technical questions about the world database, maybe others will reply here.
- Wuzzy
- Member
- Posts: 4786
- Joined: Mon Sep 24, 2012 15:01
- GitHub: Wuzzy2
- IRC: Wuzzy
- In-game: Wuzzy
- Contact:
Re: Opinons needed: Possible change in map block serializati
I think you should go ahead and fix the bug. Unless you can find a way to fix the bug without breaking compability (this would be of course the best scenario).
But if not:
Getting rid of annoying lighting bugs is always important IMO, even if you have to break compability sometimes. Bad lighting in Minetest worlds can be very annoying or frustrating.
Also, I think compability shouldn't be too big a concern at this stage. Minetest is still incomplete and has a long way to go, so introducing important features or bugfixes should be expected. I would be more concerned about compability as soon Minetest hits the 1.0.0 milestone or is in beta stage.
But PLEASE add this info in the changelog (or better: in the release announcement) when you release the next version as this is important to know. (Speaking of changelogs, I'm still waiting for the 0.4.15 changelog. :P)
Thanks for asking the community before doing important changes, however.
But if not:
Getting rid of annoying lighting bugs is always important IMO, even if you have to break compability sometimes. Bad lighting in Minetest worlds can be very annoying or frustrating.
Also, I think compability shouldn't be too big a concern at this stage. Minetest is still incomplete and has a long way to go, so introducing important features or bugfixes should be expected. I would be more concerned about compability as soon Minetest hits the 1.0.0 milestone or is in beta stage.
But PLEASE add this info in the changelog (or better: in the release announcement) when you release the next version as this is important to know. (Speaking of changelogs, I'm still waiting for the 0.4.15 changelog. :P)
Thanks for asking the community before doing important changes, however.
- paramat
- Developer
- Posts: 3700
- Joined: Sun Oct 28, 2012 00:05
- GitHub: paramat
- IRC: paramat
- Location: UK
Re: Opinons needed: Possible change in map block serializati
This PR is now being merged.
-
- Member
- Posts: 361
- Joined: Tue Jan 19, 2016 16:46
- GitHub: jdc843
- IRC: jdc843
- In-game: 843jdc
- Location: USA
Re: Opinons needed: Possible change in map block serializati
I guess it is too late to ask for a length|size field for serialized inventory in the world format? Every other data section of the database has a length|size|or count but not serialized inventory. ha. I don't even care about the contents right now. That's assuming that it is always there too. I just want to skip past it to get to the next nodes' metadata. But if people can write code that reads it and have a working game, I can read it also. IDK when though :)
- paramat
- Developer
- Posts: 3700
- Joined: Sun Oct 28, 2012 00:05
- GitHub: paramat
- IRC: paramat
- Location: UK
Re: Opinons needed: Possible change in map block serializati
Best open an issue in the Github Minetest engine webpage.
- lag01
- Member
- Posts: 321
- Joined: Sun Mar 16, 2014 03:41
- GitHub: AndrejIT
- IRC: lag01
- In-game: lag
- Contact:
Re: Opinons needed: Possible change in map block serializati
Where can i find new specifications for 27. map format?
It seems https://github.com/minetest/minetest/bl ... format.txt is for version 25.
It seems https://github.com/minetest/minetest/bl ... format.txt is for version 25.
-
- Developer
- Posts: 131
- Joined: Sat Feb 07, 2015 17:16
- GitHub: nerzhul
- IRC: nrzkt
- In-game: nrz
- Location: France
- Contact:
Re: Opinons needed: Possible change in map block serializati
paramat, if you raise the serialization version i hope we can move to zstd algorithm for serialization in database, it's a great thing
Who is online
Users browsing this forum: No registered users and 7 guests