Gestion d'exceptions#23
Conversation
| """ | ||
| Determines if the message is severe enough to trigger the shutdown procedure. | ||
| """ | ||
| return get_message_severity(message_id) <= CRITICAL_ERROR |
There was a problem hiding this comment.
the higher the message id, the more critical it is?
return message_id >= critical
this would make more sense, but you're the one that knows the message id scale meaning
There was a problem hiding this comment.
same question
| print(data) | ||
| if data.is_valid(): | ||
| check_results = perform_check(name, data.value) | ||
| if isinstance(check_results, list): |
There was a problem hiding this comment.
isinstance does not comply with coding standards, should be named is_instance or isInstance
There was a problem hiding this comment.
isinstance est une fonction native à Python
| int: The message id. | ||
| """ | ||
|
|
||
| if isinstance(device, str): |
There was a problem hiding this comment.
C'est natif à Python ça se change pas
EtienneLescarbeault
left a comment
EtienneLescarbeault
left a comment
There was a problem hiding this comment.
très clean, juste voir la logique pour require_shutdown sinon c'est bon pour moi
| """ | ||
| Determines if the message is severe enough to trigger the shutdown procedure. | ||
| """ | ||
| return get_message_severity(message_id) <= CRITICAL_ERROR |
There was a problem hiding this comment.
same question
| print(data) | ||
| if data.is_valid(): | ||
| check_results = perform_check(name, data.value) | ||
| if isinstance(check_results, list): |
There was a problem hiding this comment.
isinstance est une fonction native à Python
| int: The message id. | ||
| """ | ||
|
|
||
| if isinstance(device, str): |
There was a problem hiding this comment.
C'est natif à Python ça se change pas
3 participants
Cette PR est assez grosse et je pense qu'elle mérite d'être bien discutée. Je pense avoir bien avancé, mais il risque d'y avoir pas mal de choses à changer (soit maintenant, soit dans le futur quand on va l'utiliser / la tester).
Je me suis basé sur cette page du wiki pour l'implémenter, histoire d'avoir une manière stable de communiquer les messages avec l'interface.
J'ai dû changer quelques paramètres dans la configuration pour tout faire fonctionner, donc j'ai uniformisé un peu le code écrit depuis quelque mois en même temps.
Finalement comme demandé par le reste de l'équipe, on a en général trois niveaux d'exceptions pour les températures et la charge:
Closes: #2
Relates to: #10