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

Convert PROCESS error handling to new system #3524

Open
timothy-nunn opened this issue Feb 4, 2025 · 0 comments
Open

Convert PROCESS error handling to new system #3524

timothy-nunn opened this issue Feb 4, 2025 · 0 comments

Comments

@timothy-nunn
Copy link
Contributor

There is a new exception handling system in PROCESS, process.exceptions which (currently) contains three error types:

  • ProcessError: the base PROCESS error.
  • ProcessValidationError: errors in validation of input.
  • ProcessValueError: ValueError's raised within PROCESS.

PROCESS needs to be upgraded to this system in two ways:

  1. All of the level 3 errors in process/utilities/errorlist.json that are created from inside Python should become an appropriate exception, this may require adding additional exceptions to process.exceptions.
  2. All exceptions currently raised in Python should raise a PROCESS-specific error. This allows us to isolate errors originating from PROCESS models and those from other packages. Similarly, any dependant of us will be able to distinguish PROCESS errors.
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

No branches or pull requests

1 participant