Skip to content
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

Clear existing offset info when parking #330

Merged
merged 1 commit into from
Jan 13, 2018

Conversation

wtgee
Copy link
Member

@wtgee wtgee commented Jan 13, 2018

This is only an issue if the system temporarily parks because of bad weather then starts back up and selects the same target. Without this the mount will try to adjust tracking after taking a pointing picture.

Fix a bad log call

@wtgee wtgee requested a review from jamessynge January 13, 2018 07:48
@codecov
Copy link

codecov bot commented Jan 13, 2018

Codecov Report

Merging #330 into develop will increase coverage by <.01%.
The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #330      +/-   ##
===========================================
+ Coverage    83.81%   83.81%   +<.01%     
===========================================
  Files           51       51              
  Lines         3367     3368       +1     
  Branches       422      422              
===========================================
+ Hits          2822     2823       +1     
  Misses         404      404              
  Partials       141      141
Impacted Files Coverage Δ
pocs/state/states/default/parking.py 75% <50%> (+2.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6168ba0...90a0cd6. Read the comment docs.


pocs.next_state = 'parked'

if pocs.observatory.has_dome:
pocs.say('Closing dome')
if not pocs.observatory.close_dome():
self.logger.critical('Unable to close dome!')
pocs.logger.critical('Unable to close dome!')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doh! Thanks for catching that.

@jamessynge jamessynge merged commit 21e5408 into panoptes:develop Jan 13, 2018
@wtgee wtgee deleted the reset-offset-info branch January 13, 2018 19:32
@wtgee wtgee mentioned this pull request Jan 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants