Bugfix #405325: Keeping meridian flip in capture and mount in sync.
ClosedPublic

Authored by wreissenberger on Mar 15 2019, 8:45 PM.

Details

Summary

Correcting the meridian flip function for situations where the meridian flip has been activated after a capture sequence without the meridian flip activated has been started.

Additionally, small corrections are added:

  • Capture status display in the summary module corrected
  • Step size of the meridian flip distance in the mount module set to 1/10 - similar as in the capture module
Test Plan
  • set "Meridian Flip..." in the mount module NOT ticked
  • start a capturing session close to the meridian with "Meridian Flip..." NOT ticked in the capture module
  • wait until the mount passes the meridian
  • tick ""Meridian Flip..." in the mount module

Expected result: the meridian flip is executed as soon as the meridian has been passed and the next capture has been finished. After the meridian flip, capturing is continued.

Diff Detail

Repository
R321 KStars
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
wreissenberger created this revision.Mar 15 2019, 8:45 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMar 15 2019, 8:45 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
wreissenberger requested review of this revision.Mar 15 2019, 8:45 PM
wreissenberger edited the summary of this revision. (Show Details)Mar 15 2019, 8:49 PM

setMeridianFlipValues is not a DBus function? doesn't need to be added to the Mount.xml file?

setMeridianFlipValues is not a DBus function? doesn't need to be added to the Mount.xml file?

No, I changed based the implementation on slots and signals. Changing a value in either capture or mount sends out a signal and the other side receives it and sets the values accordingly. This seems to me a more elegant way than using calls via DBus.

I think what Jasem means is that tab modules may work with slots and signals internally, but should communicate between each other with dbus. Just as if they were independent processes, possibly distributed over different machines.

wreissenberger planned changes to this revision.Mar 17 2019, 1:49 PM

OK, understood. I will change the signal handling such that they are using the D-Bus.

wreissenberger updated this revision to Diff 54093.EditedMar 17 2019, 3:17 PM

Value changes to meridian flip setup exposed to D-Bus

So a lot of signal/slots are already connected between all the modules now. There is no direct cables between them, it only happens via signal/slots. But we should exposure DBus where it make sense as well. At any rate, is this good to go now and reliable?

At any rate, is this good to go now and reliable?

From my perspective, yes.

mutlaqja accepted this revision.Mar 17 2019, 3:53 PM
This revision is now accepted and ready to land.Mar 17 2019, 3:53 PM
This revision was automatically updated to reflect the committed changes.