chore: Add message to AssertionError

Xilinx / finn-base

Open Source Compiler Framework using ONNX as Frontend and IR

https://finn-base.readthedocs.io/

BSD 3-Clause "New" or "Revised" License

29 stars 17 forks source link

chore: Add message to AssertionError #44

Closed rpitonak closed 3 years ago

rpitonak commented 3 years ago

Add a message to Assertion Error when file passed to ModelWrapper does not exist (or is not a file).

A lot of times I pass a wrong path to ModelWrapper and cannot see the error at first glance, but just after I spot the line of code in the traceback. This PR makes this more explicit.

maltanar commented 3 years ago

Thanks for the PR @rpitonak ! This was failing the linting check (probably a spaces issue in the line you modified) so I've run the linter and pushed to your source branch. For any future PRs, can you make sure you run linting prior to submission: https://finn-dev.readthedocs.io/en/latest/developers.html#linting