Skip to content

Suggestions for igb_to_vtkhdf.py

Marie Houillon requested to merge suggestions-paraview into paraview

Here are some suggestions, I'm happy to discuss them.

In particular:

  • Add header docstring to summarize the script's functionality.
  • Mark options as "required" rather than raising an error if they are not provided: argument checking is handled by the argparse module.
  • Give examples of cell types (avoids to have to check on the VTK nomenclature for simple cases)
  • Raise SystemExit rather than FileExistsError if the user decides not to overwrite existing file. This avoids the stack traceback to be printed.

Merge request reports