-
Notifications
You must be signed in to change notification settings - Fork 24
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 example tests #34
Conversation
Commenting out the first two lines
Add more tests for more examples
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #34 +/- ##
===========================================
+ Coverage 44.78% 60.97% +16.19%
===========================================
Files 29 30 +1
Lines 1842 1858 +16
===========================================
+ Hits 825 1133 +308
+ Misses 1017 725 -292 ☔ View full report in Codecov by Sentry. |
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.
Haven't started looking into you mlx changes yet, GitHub does not help there.
Revert any change so it matches the version in main
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.
Checked mlx differences in MATLAB.
Hi folks,
I am adding some smoke-level tests for most of our examples. For the examples, I did not add a test, we may need to do some small edits. So I decided to wait.
The set-up method creates an .env file which deletes in a teardown. The .env file will contain the secret key. We also have a teardown to make sure the key environment variable is unset and another one to close any opened figures.