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

End of night shutdown #390

Closed
wtgee opened this issue Jan 19, 2018 · 2 comments · Fixed by #407
Closed

End of night shutdown #390

wtgee opened this issue Jan 19, 2018 · 2 comments · Fixed by #407

Comments

@wtgee
Copy link
Member

wtgee commented Jan 19, 2018

At the end of the night the scheduler is not finding any valid observations, which is okay, but the system sort of treats this as an error and shuts down. This triggers a "error" shut down that stops the state machine, meaning it won't come up automatically the next night. Output from logs is here:

I0118 15:41:30.891 messaging.py:122     PANCHAT Weather is good and it is dark. Something must have gone wrong. Stopping loop
I0118 15:41:30.892 machine.py:171       Stopping POCS states

See also #373

@jamessynge
Copy link
Contributor

Thanks for filling this. I noticed that message this morning and wondered what was going to happen tonight.

@wtgee
Copy link
Member Author

wtgee commented Jan 19, 2018

The problem ultimately comes from a should_retry counter that is part of core.py. This defaults to 3 and is checked in the sleeping state. It was added earlier because some minor tracking errors (a no solve-field IIRC) were causing the unit to park itself but then it would waste the rest of the night sleeping.

In this situation, if NoObservation is found early enough it will send to sleep, but will retry to start up again. After three attempts it will think something is "really" wrong so stop_states.

You will have to Ctrl-C and start again as the entire state machine has been stopped.

Could be fixed in a couple of ways.

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 a pull request may close this issue.

2 participants