X Tutup
Skip to content

Add ability to reroll random systems to System Editor#5901

Merged
sturnclaw merged 4 commits intopioneerspacesim:masterfrom
sturnclaw:editor-sysgen
Aug 25, 2024
Merged

Add ability to reroll random systems to System Editor#5901
sturnclaw merged 4 commits intopioneerspacesim:masterfrom
sturnclaw:editor-sysgen

Conversation

@sturnclaw
Copy link
Member

Continuing on from the discussion in #5892, this PR cleans up some generation bugs in binary systems which caused the orbits of stars to change once edited (due to orbital parameters not being written back to the body), and adds a new "regenerate system from seed" functionality to quickly generate new variants of the edited system. It also addresses a unit conversion issue in body radius display when using Earth Radius as a unit.

Due to data plumbing issues, regenerating a system can only be done for a full-procedurally generated system or a "partially random" system sourced from the local_stars database of systems with randomly generated contents.

2024-08-23.16-23-47.mp4

- Add two new menu options:
  1. re-generate system contents from current seed
  2. generate random seed for current system and completely regenerate contents
- These operations can only be performed on systems loaded from a procgen system due to random generation's dependence on an extant Sector::System
- Can be quite helpful to get a "scaffold" system to start building a custom system from
- Wrong values used for body radius unit conversion
- Don't render a close button on the primary editor viewport
- Ensure eccentricity and semi-major axis values are properly written back to individual bodies
@sturnclaw
Copy link
Member Author

I've tested this code to ensure it doesn't change the consistency of system layouts in bugged binary systems, so it should be compatible with old saves - not that it matters, since we're already savebumping.

@sturnclaw sturnclaw merged commit 2c962df into pioneerspacesim:master Aug 25, 2024
@sturnclaw sturnclaw deleted the editor-sysgen branch October 7, 2024 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

X Tutup