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

Merge OSVVM errors/alerts with vunit checks. #938

Closed
g0t00 opened this issue May 24, 2023 · 4 comments
Closed

Merge OSVVM errors/alerts with vunit checks. #938

g0t00 opened this issue May 24, 2023 · 4 comments

Comments

@g0t00
Copy link

g0t00 commented May 24, 2023

If I see this correctly currently when for example using OSVVM function which trigger OSVVM errors these silently get dropped when the test bench itself passes.
My use case is that i use a OSVVM Generic Scoreboard with the builtin Check function and record datatype.
But also

          AffirmIf(False, "Test");

per default does not show anywhere.
I currently work around this by calling check(GetAlertCount = 0); before test_runner_cleanup.
But I stumbled around for a while until I noticed this. My naive assumption was that this is somehow integrated, as vunit is somehow showing OSVVM support.
Can we at automatically do this check during runner cleanup. Or better merge the OSVVM alerts with the vunit checks during cleanup/reporting?

@g0t00
Copy link
Author

g0t00 commented May 24, 2023

Just realized that #776 is about this...

@LarsAsplund
Copy link
Collaborator

My intention is to release such an interface for VUnit that would enable you to pipe VUnit logs into OSVVM. I've also prototyped the changes needed to OSVVM as a proof of concept. I will probably create a PR of that work but in the end it is for the OSVVM community to maintain such a feature if there is an interest. You could also open an issue with them to get the discussion going on both ends.

@LarsAsplund
Copy link
Collaborator

I will close this in favor of #776.

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

No branches or pull requests

3 participants