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 Tests on M1 #839

Merged
merged 3 commits into from
Aug 21, 2022
Merged

Fix Tests on M1 #839

merged 3 commits into from
Aug 21, 2022

Conversation

ccoffrin
Copy link
Member

@odow, these appear to be the only changes that need to be made. Appears to be a slight numerical difference in Julia's builtin det function. I am going to wait to merge to get your input to do a work around or not in this case.

@codecov
Copy link

codecov bot commented Aug 20, 2022

Codecov Report

Merging #839 (4c5495c) into master (09bf3de) will not change coverage.
The diff coverage is n/a.

❗ Current head 4c5495c differs from pull request most recent head a6568bc. Consider uploading reports for the commit a6568bc to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #839   +/-   ##
=======================================
  Coverage   93.88%   93.88%           
=======================================
  Files          42       42           
  Lines        9882     9882           
=======================================
  Hits         9278     9278           
  Misses        604      604           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09bf3de...a6568bc. Read the comment docs.

Copy link
Collaborator

@odow odow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems unrelated to M1. It could happen on any machine with different BLAS etc.

@ccoffrin ccoffrin merged commit cbce397 into master Aug 21, 2022
@ccoffrin ccoffrin deleted the m1-test branch August 21, 2022 23:05
@odow
Copy link
Collaborator

odow commented Aug 21, 2022

FWIW, I just reproduced these two failures as well.

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