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

Allow growified residentials to be redeveloped #7

Merged
merged 40 commits into from
Jul 13, 2020
Merged

Conversation

sebamarynissen
Copy link
Owner

After the discovery that the SimGrids need to be updated in order for "generated" zones to be growable, we've applied this to the growify command as well which now means that they can be redeveloped and built over as well.

It's now possible to plop arbitrary lots anywhere on the map. Still, we're having some troubles with the rotations, so those will still need to be fixed.
We now use the binary-search-bounds module instead of the binary-search module as it exposes more useful methods such as le and lt etc.
A method has been added to free up the memory a DBPF is taking up. Useful for indexing a large amount of DBPF files: we can simply unload the ones we no longer need.
The FileIndex now uses a LRU cache which properly frees up memory for DBPF files depending on how often the entries in it are accessed. Note that you should make sure to provide sufficient memory, otherwise it will slow you down tremendously. Let's hope the cache here doesn't get in the way and provides proper memory management.
We're finally able to insert the base textures as well. Nice.
We now create the required entries in the city dbpf when they don't exist yet if we're reading in the city. We've verified that it works properly on a large city tile.
@sebamarynissen sebamarynissen self-assigned this Jul 13, 2020
@sebamarynissen sebamarynissen merged commit 257b380 into master Jul 13, 2020
@sebamarynissen sebamarynissen deleted the feature/plop branch July 13, 2020 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant