Closed pedropaulofb closed 1 year ago
After solving the reported problems, we must review the software's documentation, especially:
Please adjust the text of each argument (argparse's help=
) to correspond to the texts available in the documentation. I already updated them there.
Cases 1 (Default values) and 3 (Report generation) solved in commit f94205b5129ec1359f2efae649419c93990d5c6c
Issue 29 opened for case 2.
I identified three situations regarding arguments that we must correct in OntCatOWL:
1) Default values
The only mandatory argument that must be provided for the OntCatOWL execution is the path to the input file. When the user provides no value for the arguments, the default values must be:
This must be informed to the user in the argparse help as:
help="The loaded ontology is an incomplete model.")
help="The loaded ontology is an incomplete model (default).")
The example shows the incomplete value. We must do the same to the interactive value.
The module to be changed is the
initialization_arguments.py
.2) Reports generated in external folder
When the user explicitly passes the arguments for the OntCatOWL execution, the final report is printed as follows:
./reports
(inside the project's folder)However, if the user provides no argument and the default arguments are used, the OntCatOWL generates the report into the incorrect folder:
../reports
(inside the project's parent folder)The module to be changed is the
report_printer.py
.3) Report generation
The automatic argument value is being printed incorrectly in the report file. Please do the following modification:
f"* Automatic execution: {not configurations['is_automatic']}\n" \
f"* Automatic execution: {configurations['is_automatic']}\n" \
(remove the not token)The module to be changed is the
report_content.py
.