-
Notifications
You must be signed in to change notification settings - Fork 6
add tolerances #79
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
add tolerances #79
Conversation
83e3885 to
e0be3e0
Compare
|
/runtests |
|
Test Result: failure (details) |
|
Are you able to see what the issue would be in the asics test? There are not enough details. Possibly we should add |
|
/runtests |
|
Test Result: failure (details) |
|
/runtests |
|
Test Result: failure (details) |
|
@jacobmerson I submitted a PR to your branch with a few small changes jacobmerson#1. The fix for the failing test in self-hosted CI was pushed to main (3b5125e). |
tolerances
|
/runtests |
|
Test Result: failure (details) |
|
I'm not sure what is happening with the self-hosted CI failing; this commit (3b5125e) should have fixed the element count check. I'll fix it in another PR if needed. |
This provides tolerances that should work with PCMS and lets the user specify. Addresses #69