doc: update docstring and tesseract input/output handling #412
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Relevant issue or PR
Description of changes
1. Fix docstring
Function
sample_points_with_spheresdid not adhere to the docstring format indemo/_showcase/ansys-qoi/qoi_dataset/scripts/process/utils.py. This PR unifies docstring format.2. Input/output handling
InputFileReferenceandOutputFileReferencewhen possible. When Tesseract contains a folder path as input value, these variables are still consideredstr. Changing this would require a change of logic, which could be considered in future improvementsevaluatefunction, and in order to work with--input_pathand--output_path, the following lines have been implemented to buildPathvariables and enable writing of files to local disk.3. Update README with information on how to train with gpu
Testing done
CPU execution
qoi_dataset/main.pyqoi_train/main.pyqoi_inference/main.pyworkflow.pyGPU execution
qoi_dataset/main.pyqoi_train/main.pyqoi_inference/main.pyworkflow.py