calyx
calyx copied to clipboard
calyx-py: add line numbers to group enables, similar to Dhalia
@ayakayorihiro
Thanks a lot for this, it's great to see the source mapping here too!! We definitely need a indication of what file the code is coming from... but very exciting!!!!
@ekiwi what needs to be done to get this merged?
@ekiwi what needs to be done to get this merged?
There are mainly three blockers:
- We need to define a way to include the filename in addition to the line number.
- There is a bug where we end up with two line numbers for a single calyx
invoke, I have to spend some time tracking that down. - We need to spend some more time thinking about which calyx constructs should be annotated.
1+3 could be addressed in a future PR. 2 definitely has to be fixed before I am going to move this out of draft.
This pull request has not seen activity in 14 days and is being marked as stale. If you're continuing work on this, please reply and state how to get this PR in a mergeable state or what issues it is blocked on. If the PR is not ready for review, please mark it as a draft. The PR will be closed in 7 days if there is no further activity.