-
Notifications
You must be signed in to change notification settings - Fork 620
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
Bump Barstools #332
Bump Barstools #332
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@davidbiancolin It looks like CI is failing on not finding a |
Well this is mystifying, how did CI pass before? |
@davidbiancolin found the error, unfortunately firesim was reverted in this commit on accident: 443be01. I rebumped FireSim to a new |
Alright so the error in the midasexample is a known issue and is resolved here. We have a couple options. Either you can bump to the same commit you used before, or i can pull the test fix out into a separate PR in firesim, we can merge that, and you can bump again. |
Fixed the single test case in FireSim and re-bumped according to @davidbiancolin's request |
This is bumping
barstools
to have a couple of macro-compiler bug fixes and cleanup.