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

catch hopper error on shape so shape doesn't die #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timsainb
Copy link
Member

@timsainb timsainb commented Mar 6, 2020

the only problem with this is that error with shape could go completely unnoticed in this case...

the only problem with this is that error with shape could go completely unnoticed in this case...
@theilmbh
Copy link
Member

theilmbh commented Mar 6, 2020

That sounds like a big error.
It's better to have shape die than have the bird not be able to get food.

@timsainb
Copy link
Member Author

timsainb commented Mar 6, 2020

Shape dying will also go unnoticed because we don't get email alerts anymore and we have a chron job auto restarting. IDK how to fix this but we really need a way to notify when this happens

@theilmbh
Copy link
Member

theilmbh commented Mar 6, 2020

Well, shape dying is something you can check on periodically. What's the point of having a hopper error if we just ignore it? How can you make this notify us?

@timsainb
Copy link
Member Author

timsainb commented Mar 6, 2020

In main behavior we catch the error and add an error to the behavior CSV. Previously on shape we had an email alert. But since we moved to magpis we can't send emails afaik

@timsainb
Copy link
Member Author

timsainb commented Mar 6, 2020

I could make rpioperantctl email you when it reboots a behavior

@timsainb
Copy link
Member Author

timsainb commented Mar 6, 2020

The other issue is if the behavior dies when the hopper is up, it will stay on for longer than intended

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.

None yet

2 participants