This repository was archived by the owner on Jun 11, 2025. It is now read-only.
Static Solar System #41
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
saraazevedolopes
force-pushed
the
SolarSystem
branch
from
1904f39 to
0f873f8
Compare
saraazevedolopes
force-pushed
the
SolarSystem
branch
from
be6b2eb to
b113478
Compare
saraazevedolopes
requested review from
Mariana-rc01,
a104437ana and
voidbert
and removed request for
Mariana-rc01,
a104437ana and
voidbert
Mariana-rc01
approved these changes
Mar 15, 2025
a104437ana
approved these changes
Mar 17, 2025
voidbert
suggested changes
Mar 22, 2025
| camera->InsertEndChild(projection); | ||
| } | ||
|
|
||
| void SolarSystem::generateSolarSystem(float sceneScale, |
Owner
There was a problem hiding this comment.
This can be heavily simplified but I'm not gonna get worked up about it. Let's do what we can with the time we have. Also, in the future, we can think of implementing a group hierarchy which would imply some substantial changes, which we can leave for later.
Owner
|
Now I know where some of my action minutes have gone 😅
|
saraazevedolopes
force-pushed
the
SolarSystem
branch
2 times, most recently
from
d234828 to
10cd653
Compare
voidbert
approved these changes
Mar 22, 2025
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Solar System Hierarchy:
Solar System (Group)
│── Sun (Root)
│ │── Planet 1 (Orbits Sun)
│ │ │── Moon 1 (Orbits Planet 1)
│ │ │── Moon 2 (Orbits Planet 1)
│ │── Planet 2 (Orbits Sun)
│ │ │── Moon 1 (Orbits Planet 2)
│ │── Planet 3 (Orbits Sun)
│ │ │── Rings (Belongs to Planet 3)
│ │── Asteroid Belt (Orbits Sun)
The user can make changes to the solar system.
sceneScale – Defines the overall scale of the scene.
sunSizeFactor – Controls the size of the Sun.
planetSizeFactor – Controls the size of the planets.
moonSizeFactor – Controls the size of the moons.
distanceFactor – Controls the distance between celestial bodies.
asteroidBeltDensity – Defines the density of the asteroid belts.