Issue1177

Title Revise exit codes for malformed input files
Priority wish Status chatting
Superseder Nosy List Claudia, jendrik, malte, salome
Assigned To Keywords
Optional summary

Created on 2025-02-14.11:17:08 by Claudia, last changed by malte.

Messages
msg11798 (view) Author: malte Date: 2025-02-14.15:50:19
The translator never actively returns exit code 30. The reason for the way this is currently documented is that the driver replaces all "unexpected" translator errors/crashes by exit code 30.

Before we had actual error checking in the translator, PDDL errors would crash the translator (assertion failures or other uncaught exceptions), and this caused an error code of 30. So exit code 30 was never intended to signal bad PDDL, but rather it signals crashes, and bad PDDL caused crashes. Now that we say the translator is responsible for checking the PDDL input, such crashes are considered bugs, and hence the documentation of this exit code should be changed.

In a perhaps similar vein, the meaning of TRANSLATE_INPUT_ERROR was changed from "usage error" to "usage error or invalid PDDL" accidentally. The new error checking code used the "usage error" exit code for wrong PDDL input and was merged without much review, so we didn't notice this. We then changed the documentation to "usage error or invalid PDDL" much later when we realized that this exit code is now also used in these scenarios.
msg11792 (view) Author: Claudia Date: 2025-02-14.11:17:08
Currently exit codes 31 TRANSLATE_INPUT_ERROR and 33 SEARCH_INPUT_ERROR are overloaded. They are both used either if the command line options are wrongly specified or if the input PDDL / SAS+ files are malformed.

We would like to separate wrong command line options and malformed input files into different exit codes.

Note that on the Wiki we currently state that exit code 30 TRANSLATE_CRITICAL_ERROR can also be used for malformed PDDL input. We should check if this is the case and if so change it to the new exit code.
History
Date User Action Args
2025-02-14 15:50:20maltesetstatus: unread -> chatting
messages: + msg11798
2025-02-14 11:17:08Claudiacreate