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

PriorityQueue tests failing seemingly at random #46

Open
dobios opened this issue Oct 10, 2022 · 4 comments
Open

PriorityQueue tests failing seemingly at random #46

dobios opened this issue Oct 10, 2022 · 4 comments
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@dobios
Copy link
Member

dobios commented Oct 10, 2022

The functionality tests for the priority queue have each been failing seemingly at random from one run to the next.
Sometimes MemoryTest fails, sometimes PriorityQueueTest fails, and sometimes they both pass...

This should be really made consistent, as it either shows that the functionality of the PriorityQueue is wrong, or that the test is wrong.

@tjarker Please look into this.

@dobios dobios added the bug Something isn't working label Oct 10, 2022
@tjarker
Copy link
Member

tjarker commented Oct 12, 2022

will do!

@hansemandse
Copy link
Member

Having been over the code quite thoroughly now, I believe this issue stems from calling scala.util.Random directly without a seed instead of instantiating and initializing one for subsequent use. As this is done on several occasions (in MemoryTest.scala, for example), the order the tests are executed in may affect whether they fail or not.

Unfortunately, a similar thing is done within the chiselverify.assembly component where fixing it might be a bit more troublesome.

@dobios dobios closed this as completed Mar 26, 2024
@hansemandse
Copy link
Member

Why was this closed? The issue still persists and it doesn't look great to keep having random red crosses in PRs and on the main branch.

@dobios dobios reopened this Mar 29, 2024
@dobios
Copy link
Member Author

dobios commented Mar 29, 2024

I see that now, sorry I thought this had been solved

@dobios dobios added the help wanted Extra attention is needed label May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants