Check status after script/command execution
Open
Nobody has claimed this yet.
enhancement
good first issue
- Dominant language
- Python
- Stars
- 1.1k
- Forks
- 308
- Avg merge
- 3d 9h
- Merged PRs (30d)
- 14
Description
Many transformations and utility functions in FINN call external commands such as g++, vivado and vivado_hls without checking the return status. If the called command fails, the FINN transformation won't be aware of this. Assertions should be introduced on the return status whenever an external command is called.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating FINN transformations and utility functions that invoke external commands such as g++, vivado, and vivado_hls. Inspect how those calls currently handle their return status. Done means each relevant external command checks its status and reports failure to the transformation instead of continuing silently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100