setObserverLocation re-enables landscape, when setFlagLandscape false

Bug #1783752 reported by Phil Shepherd
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Stellarium
Fix Released
Undecided
Alexander Wolf

Bug Description

Stellarium version 0.18.1 :

I have a short script showing a quick trip to the Moon.

- Initial landscape set on
- set alt / az view
- pause
- setFlagLandscape(false)
- setObserverLocation using a duration animation of 10 seconds to show travel to the Moon

The landscape appears as soon as motion starts.

example script attached

Phil

Revision history for this message
Phil Shepherd (psjshep) wrote :
Revision history for this message
Phil Shepherd (psjshep) wrote :

btw, I have tried all sorts of combinations... each time, the landscape appears as soon as setObserverLocation is called...

fyi

Changed in stellarium:
assignee: nobody → Alexander Wolf (alexwolf)
status: New → Confirmed
Revision history for this message
Alexander Wolf (alexwolf) wrote :

Fixed auto-enabling landscapes when setObserverLocation is call: https://github.com/Stellarium/stellarium/commit/c689edd4d6798dd828547fdb3526cdadc2bd25d3

I've renamed option "Auto-enabling for the atmoshere" to the "Auto-enabling for the environment" and enhance the behavior of this feature to the landscapes too. By default this option is enabled in the GUI and end-user will be see no changes in behavior.

In scripts this feature can be disabled throug calling special method:
LandscapeMgr.setFlagEnvironmentAntoEnabe(false);

Work-around for previous versions of Stellarium (0.18.0-0.18.1 and maybe 0.17.x) is simple: call LandscapeMgr.setFlagLandscape() after calling core.setObserverLocation();

Methods LandscapeMgr::setFlagAtmosphereAutoEnable() and LandscapeMgr::getFlagAtmosphereAutoEnable() marked as deprecated and keep in the code for backward compatibility. Both deprecated methods will be removed in next series.

Changed in stellarium:
status: Confirmed → Fix Committed
Revision history for this message
Phil Shepherd (psjshep) wrote :

Nice 1, Alexander... thanks

Just a quick question about the feature enhancement:

Will the ability to seperately set/unset the atmostphere and landscape still be available?

I sometimes show the effect of atmosphere & light polution by switching off the atmosphere - highly effective.

Phil

Revision history for this message
Phil Shepherd (psjshep) wrote :

o, & when will these changes be in the next Beta?

Phil

Revision history for this message
Alexander Wolf (alexwolf) wrote :

> Will the ability to seperately set/unset the atmostphere and landscape still be available?

Yes, of course.

New method LandscapeMgr::setFlagEnvironmentAutoEnabe(); just enables or disables the automatic enabling of atmosphere, fog and landscape when location is changed.

Revision history for this message
Phil Shepherd (psjshep) wrote :

thx

Revision history for this message
Phil Shepherd (psjshep) wrote :

When will these changes make their way into a Windows build? I am not able to compile for that.

Thanks,
Phil

Revision history for this message
gzotti (georg-zotti) wrote :

Alexander tirelessly makes betas for all platforms almost every weekend.

Revision history for this message
Phil Shepherd (psjshep) wrote :

... and very much appreciated it is

Thanks

Phil

Revision history for this message
Phil Shepherd (psjshep) wrote :

I have successfully tested my scripting with the new functionality - works as described...

Thank you...

When is it likely to appear in the next production release?

Phil

Revision history for this message
Alexander Wolf (alexwolf) wrote :

The next release will be in few next days, maybe week.

gzotti (georg-zotti)
Changed in stellarium:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.