Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix VCD generation from Verilator model #1341

Merged
merged 2 commits into from
Aug 24, 2023
Merged

Fix VCD generation from Verilator model #1341

merged 2 commits into from
Aug 24, 2023

Conversation

pcotret
Copy link
Contributor

@pcotret pcotret commented Aug 22, 2023

This pull request is an answer to my issue described at #1340
Quick comments:

  • I've removed the -j${NUM_JOBS} in Verilator generation as it basically shut down my VM (8 cores...).
  • Modifications in Readme: I've tested with Verilator v4.110. I assume it should be compatible with other versions as well.

@github-actions
Copy link
Contributor

❌ failed run, report available here.

@JeanRochCoulon
Copy link
Contributor

Hello @pcotret
Nice to get your contribution
The CI is currently red. I think it is needed to rebase your PR to pass the CI. Can you ?

@pcotret
Copy link
Contributor Author

pcotret commented Aug 23, 2023

I'll try it!

@github-actions
Copy link
Contributor

❌ failed run, report available here.

@pcotret
Copy link
Contributor Author

pcotret commented Aug 23, 2023

CI goes red again, only for the "Environment check" step.
Is there a way to see exactly what is done in this step? (I assume it may be related to the environment variable I've modified)

@github-actions
Copy link
Contributor

❌ failed run, report available here.

1 similar comment
@github-actions
Copy link
Contributor

❌ failed run, report available here.

@JeanRochCoulon
Copy link
Contributor

I suspect a misfunctionality of the CI. I relaunched.

@github-actions
Copy link
Contributor

✔️ successful run, report available here.

@JeanRochCoulon
Copy link
Contributor

Now it is GREEEEN. Thanks for the contribution. I merge.
Have fun with CVA6 !

@JeanRochCoulon JeanRochCoulon merged commit 9d6e434 into openhwgroup:master Aug 24, 2023
6 checks passed
@pcotret pcotret mentioned this pull request Aug 24, 2023
1 task
@Gchauvon Gchauvon mentioned this pull request Aug 29, 2023
1 task
Abdulwadoodd pushed a commit to 10x-Engineers/cva6 that referenced this pull request Sep 15, 2023
* tools/vptool/vptool/vp.py (OrderedDict): Import from 'collections', not
  from 'typing'.

Signed-off-by: Zbigniew Chamski <[email protected]>
Abdulwadoodd pushed a commit to 10x-Engineers/cva6 that referenced this pull request Sep 15, 2023
…rect-import

Fix incorrect Python import (fixes issue openhwgroup#1341.)
Abdulwadoodd pushed a commit to 10x-Engineers/cva6 that referenced this pull request Sep 15, 2023
* tools/vptool/vptool/vp.py (OrderedDict): Import from 'collections', not
  from 'typing'.

Signed-off-by: Zbigniew Chamski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants