Skip to content

Useful observer orientation#3033

Merged
gzotti merged 34 commits into
masterfrom
useful-observer-orientation
Feb 7, 2023
Merged

Useful observer orientation#3033
gzotti merged 34 commits into
masterfrom
useful-observer-orientation

Conversation

@gzotti
Copy link
Copy Markdown
Member

@gzotti gzotti commented Jan 30, 2023

Description

After @10110111 discovered an important detail which improved orientation of the "observer" pseudo-planets (#3029), we can finally make them a bit more useful. Notable changes:

  • when on an observer, freeze location on the North pole
  • inhibit landscape drawing
  • change location name display in the bottom bar
  • don't draw a few coordinate grids (more? please comment)
  • Modify Location dialog to inhibit a few settings while in observer mode
  • update SUG

Fixes #1202 (issue)

Screenshots (if appropriate):

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • This change requires a documentation update

How Has This Been Tested?

Test Configuration:

  • Operating system: Win11
  • Graphics Card: Geforce M3070Ti

Checklist:

  • My code follows the code style of this project.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (header file)
  • I have updated the respective chapter in the Stellarium User Guide
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@gzotti gzotti added the enhancement Improve existing functionality label Jan 30, 2023
@gzotti gzotti self-assigned this Jan 30, 2023
@gzotti gzotti added this to the 23.1 milestone Jan 30, 2023
@github-actions github-actions Bot requested review from 10110111 and alex-w January 30, 2023 18:50
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 30, 2023

Great PR! Please pay attention to the following items before merging:

Files matching src/**/*.cpp:

  • Are possibly unused includes removed?

Files matching guide/**:

  • Did you remember to update screenshots to match new updates?
  • Did you remember to grammar check in changed part of documentation?

This is an automatically generated QA checklist based on modified files.

Comment thread src/core/StelLocation.hpp Outdated
@10110111
Copy link
Copy Markdown
Contributor

don't draw a few coordinate grids (more? please comment)

What other grids do we have? Azimuthal grid should remain IMO. Equatorial is now equivalent to it (except for the units of azimuth), so can be disabled indeed. What the fixed equator line is I don't know (just a line on the ground?)

@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Jan 30, 2023

don't draw a few coordinate grids (more? please comment)

What other grids do we have? Azimuthal grid should remain IMO. Equatorial is now equivalent to it (except for the units of azimuth), so can be disabled indeed. What the fixed equator line is I don't know (just a line on the ground?)

The fixed equatorial system is the system of hour angle/declination used in classical position astronomy, i.e. does not make sense in a location-less place. Likewise, the horizon/compass marks are useless. Others may be discussed. Just that I need to play a bit more now. It seems the displayed "azimuth" grid is now finally really aligned with the planet's axis.

Comment thread src/core/modules/Orbit.hpp Outdated
@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Jan 31, 2023

Now that we can really see Jupiter and Saturn systems more stabilized, I can see (again) that the moons' orbit lines still overdraw the planets when the respective moon is closer than the planet. (#82)

@10110111

This comment was marked as outdated.

@10110111

This comment was marked as outdated.

@gzotti

This comment was marked as outdated.

@10110111

This comment was marked as outdated.

Comment thread data/ssystem_major.ini Outdated
@gzotti

This comment was marked as resolved.

Comment thread data/ssystem_major.ini Outdated
Comment thread data/ssystem_major.ini Outdated
@10110111

This comment was marked as resolved.

Comment thread src/gui/LocationDialog.cpp Outdated
@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 5, 2023

OK, should be ready for a test. The wheeling through now fails, though.

@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 6, 2023

Huh, I never did that... You have always new ways of operation! :-)

@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 6, 2023

Grr, it's simple if Location panel has been created already before searching.
OK, that was not so hard. But going from Jupiter observer to Saturn shows the same issue as before: default landscape for Saturn does not load. I am afraid I have to add an optional parameter to StelCore::moveObserverTo(., ., ., landscapeID) which is then emitted in an enlarged signal StelCore::targetLoationChanged(StelLocation, QString)

@alex-w
Copy link
Copy Markdown
Member

alex-w commented Feb 7, 2023

@gzotti please check another "new way of operation" :)

  1. Run Stellarium
  2. Enable showing nomenclature
  3. Open Search tool and find any nomenclature item
  4. Ctrl+G (Go to nomenclature item)
  5. Open Location dialog

What happenes at step 4 and 5?

@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 7, 2023

@gzotti please check another "new way of operation" :)
[...]

What happenes at step 4 and 5?

It works for me. What did you see?

@alex-w
Copy link
Copy Markdown
Member

alex-w commented Feb 7, 2023

It works for me. What did you see?

Good news! I didn't checked it - I just remembered about this behaviour and shared notice about, but in that time I didn't can check it.

- also removed a few unneeded headers
@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 7, 2023

@10110111 do you still see anything that should be corrected?
I think this time I will only rebase, not squash, despite some experimental commits.

@10110111
Copy link
Copy Markdown
Contributor

10110111 commented Feb 7, 2023

Functionally I didn't notice any more glitches. Code-wise though, cannot help but object to the committing of commented-out code. It's especially ugly when you doubly comment-out (see the body of LocationDialog::handleDialogSizeChanged), or introduce fresh-but-commented-out code (as in LocationDialog::setLocationUIvisible).

@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 7, 2023

Ah yes, thanks for reminding me.

@gzotti
Copy link
Copy Markdown
Member Author

gzotti commented Feb 7, 2023

Shall I also clean out ssystem_major.ini? Then going back to the old behaviour is more difficult. But maybe it's even time to no longer go back! (remove the define in Orbit.h) @alex-w , @10110111 ?

@alex-w
Copy link
Copy Markdown
Member

alex-w commented Feb 7, 2023

Clean up the ssystem_major.ini file is OK for me

@gzotti gzotti merged commit 935dfd5 into master Feb 7, 2023
@gzotti gzotti deleted the useful-observer-orientation branch February 7, 2023 19:05
@github-actions
Copy link
Copy Markdown

Hello @gzotti!

The enhancement or feature has been merged into source code and you may test it via building Stellarium from source code or wait the weekly development snapshot...

@github-actions
Copy link
Copy Markdown

Hello @gzotti!

The fix has been merged into source code and you may test it via building Stellarium from source code or wait the weekly development snapshot...

@alex-w alex-w added state: published The fix has been published for testing in weekly binary package and removed state: fixed labels Mar 13, 2023
@github-actions
Copy link
Copy Markdown

Hello @gzotti!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

@alex-w alex-w removed the state: published The fix has been published for testing in weekly binary package label Mar 27, 2023
@github-actions
Copy link
Copy Markdown

Hello @gzotti!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improve existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Solar System Observer point not above the Sun?

3 participants