Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Submap Biome generation #1151

Open
CASALK opened this issue Oct 11, 2024 · 9 comments
Open

Submap Biome generation #1151

CASALK opened this issue Oct 11, 2024 · 9 comments
Assignees
Labels

Comments

@CASALK
Copy link

CASALK commented Oct 11, 2024

I was going to create a small submap to represent a mirrored version of the world I had already made, but the map regenerated the biomes and precipitation. Despite what the message before committing to the submap stating, the biomes and precipitation would be copied, which did not happen. This is going from Mamou map to Matei map.

Steps to reproduce

  1. Open the Mamou map, and zoom into the Gateway marker near the top of the Swangulfian empire(the big one on the left).
  2. Zoom in just enough to get the two lakes on the right side, that the Gateway is about in the center.
  3. Only turn on biome layer. And then go to tools, and click submap. Use the base checkmarks.

The expectation is that the submap would just be a mini version of what I had on the main map, biomes and precipitation included.

.map file
Mamou 2024-10-10-21-30.zip
Matei 2024-10-10-22-05.zip

Screenshots
Screenshot 2024-10-10 222209
Screenshot 2024-10-10 222140

System

  • Version: v1.105.12
  • Browser: Firefox
  • OS: Windows 10
@goteguru
Copy link
Collaborator

Confirmed. Submap do NOT copy biomes. It actually regenerates biome data, in order to get better biome resolution. Biome type is calculated deterministically based on temperature and perception. (At least that was the case in previous version of AFMG).

It seems the real problem is that the temperature import has some kind of bug. I'll check it.

@Azgaar
Copy link
Owner

Azgaar commented Oct 11, 2024

I will do some changes in the function description and also working on some refactoring. Let me know if you have some changes, it would be nice to avoid conflicts :)

@goteguru
Copy link
Collaborator

No change yet, still trying to identify the root of the problem. I can not replicate the issue with newly generated maps. The problem is with that specific map (Mamou). It seems the cell data got corrupted somehow.

If I simply do "submap" on the whole map (ie. remapping it 1:1) the temperature map changes completely. Strange.

@CASALK additional info about the history of the map would be welcome (transformations, possible prior submaps and/or original version, export/import)

@Azgaar
Copy link
Owner

Azgaar commented Oct 11, 2024

  • Cell data corruption - I know it still happens, some map have cells number mismatch, reasons are yet to be found
  • Temperature change - I believe it's due to the 'converting map position on the planet' part. There were changed from my side when I have added latitude input, but it looks that now it works incorrectly

@Azgaar
Copy link
Owner

Azgaar commented Oct 11, 2024

Current changes from my side are significant, but I just started to look at the core resampling process, I've never analysed this code in details before :)

@goteguru
Copy link
Collaborator

goteguru commented Oct 11, 2024

Ah, ok. Which one is the stable branch? (the one azgaar.github.io is running on)? Is it the master?
I can't see the 'converting map position on the planet' part...

@Azgaar
Copy link
Owner

Azgaar commented Oct 11, 2024

The current is master. Here is the link to this part: https://github.com/Azgaar/Fantasy-Map-Generator/blob/master/modules/ui/submap.js#L240

@CASALK
Copy link
Author

CASALK commented Oct 12, 2024

@goteguru Thanks for taking a look! The older map (Mamou), I made on Feburary 12, 2023. Ive been updating it as I've reshaped the lore and states around to best fit what I want. I still have every version of map that I made, I've only made one other submap, which does copy the biome and temperature properly. I can share that file once I have access to my desktop. The biggest changes I've only made were to states and such, nothing touching biome or temperature. Nothing with elevation either.

@CASALK
Copy link
Author

CASALK commented Oct 13, 2024

Monia 2024-09-19-22-55.zip

Here is the other submap I have made, which was made in early-mid September. The biomes match the location of the original.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants