-
-
Notifications
You must be signed in to change notification settings - Fork 792
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
Move some solar eclipse computations from AstroCalcDialog #3747
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files. |
33fee57
to
d88c63f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good improvement, thanks!
But please add new file (src/core/SolarEclipseComputer.cpp
) into po/stellarium/POTFILES.in
also and remove extra empty lines.
The empty line at the end? Or is there some more? |
d88c63f
to
264d1fc
Compare
See CodeFactor report please |
264d1fc
to
998e4bc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, good idea to split this huge class.
998e4bc
to
bd1ffca
Compare
Let's wait until the CI finishes, after which, if successful, I'll merge. |
This reduces the size of AstroCalcDialog.cpp by 15% while localizing much of solar eclipse map generation and some other related computations in another file.
bd1ffca
to
af75ecd
Compare
Hello @10110111! Please check the fresh version (development snapshot) of Stellarium: |
This reduces the size of AstroCalcDialog.cpp by 15% while localizing much of solar eclipse map generation and some other related computations in another file.