Skip to content

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

Open
timsainb wants to merge 1 commit into
masterfrom
fix-hopper-error-on-shape
Open

catch hopper error on shape so shape doesn't die#127
timsainb wants to merge 1 commit into
masterfrom
fix-hopper-error-on-shape

Conversation

@timsainb

@timsainb timsainb commented Mar 6, 2020

Copy link
Copy Markdown
Member

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

theilmbh commented Mar 6, 2020

Copy link
Copy Markdown
Member

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

@timsainb

timsainb commented Mar 6, 2020

Copy link
Copy Markdown
Member Author

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

theilmbh commented Mar 6, 2020

Copy link
Copy Markdown
Member

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

timsainb commented Mar 6, 2020

Copy link
Copy Markdown
Member Author

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

timsainb commented Mar 6, 2020

Copy link
Copy Markdown
Member Author

I could make rpioperantctl email you when it reboots a behavior

@timsainb

timsainb commented Mar 6, 2020

Copy link
Copy Markdown
Member Author

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.

2 participants