Issue #48: Replace deprecated .graph.input/output[0].name with get_global_in/out methods

[rothej]

Added unit tests in tests/util/test_modelwrapper.py. All pre-commit hooks pass, new unit tests verify the helper methods works

[rothej]

Using ./run-docker.sh quicktest, results are:

= 1163 passed, 16 skipped, 5 xfailed, 1 xpassed, 77462 warnings in 66.39s (0:01:06) =

All instances of old pattern were replaced with the new pattern in the codebase.

[auphelia]

Hi @rothej,
Thank you for picking up this open issue! This issue dates back to when the modelwrapper (https://github.com/fastmachinelearning/qonnx/blob/main/src/qonnx/core/modelwrapper.py) was still part of FINN. As you can see from the link, it’s now part of qonnx.

Since it is defined in qonnx, the usual workflow is:

• Add the new methods in qonnx.
• Update the FINN commit (https://github.com/Xilinx/finn/blob/dev/fetch-repos.sh) to use the updated version of qonnx.
• Adjust the code accordingly, here you can use what you have already done in your PR.

Would you mind following that structure and first creating a PR to qonnx main that adds those new class methods, instead of adding them on top of modelwrapper inside FINN? I’m sure other users would also appreciate having those new class methods available directly in qonnx.

[rothej]

@auphelia not a problem! I'll go ahead and do that