-
Notifications
You must be signed in to change notification settings - Fork 4
102 plot decision #136
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
102 plot decision #136
Conversation
Code Coverage Summary
Diff against main
Results for commit: 3afebef Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Test Performance Difference
Additional test case details
Results for commit 668880f ♻️ This comment has been updated with latest results. |
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Daniel Sabanes Bove <[email protected]> Co-authored-by: 27856297+dependabot-preview[bot]@users.noreply.github.com <27856297+dependabot-preview[bot]@users.noreply.github.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Daniel Sabanes Bove <[email protected]> Co-authored-by: 27856297+dependabot-preview[bot]@users.noreply.github.com <27856297+dependabot-preview[bot]@users.noreply.github.com>
Hi @danielinteractive before I do snaps, could you kindly review this plot from the example file please ? I used the same colours as Edit, the last previous version here Edit, will add "%" |
Unit Tests Summary 1 files 20 suites 6m 12s ⏱️ Results for commit 3afebef. ♻️ This comment has been updated with latest results. |
Looks good @audreyyeoCH ! |
thanks @danielinteractive, it's ready for another review |
yaaaaaaaayyyy 🥳 |
close #102