-
Notifications
You must be signed in to change notification settings - Fork 0
[Fix] Use multisession plan in parallel plotting test #144
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #144 +/- ##
==========================================
+ Coverage 86.51% 87.24% +0.73%
==========================================
Files 65 66 +1
Lines 1379 1435 +56
==========================================
+ Hits 1193 1252 +59
+ Misses 186 183 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
This PR adds a new test file to exercise parallel plotting utilities using the multisession plan for cross-platform compatibility. The test ensures proper worker teardown and validates the multicore functionality in plotting contexts.
Key changes:
- Adds comprehensive parallel plotting test coverage using
multisessionplan - Implements proper worker cleanup with
withr::defer - Ensures cross-platform compatibility by avoiding platform-specific
multicoreplan
Co-authored-by: antoniojbt <[email protected]>
Co-authored-by: antoniojbt <[email protected]>
…e-9c18-c17e0da1b981 Fix parallel test failures by correcting worker detection timing
What was changed and why
tests/testthat/test-plot-parallel.Rto exercise parallel plotting utilities using a cross-platformmultisessionplan.future::plan("sequential")viawithr::defer.Tests added
tests/testthat/test-plot-parallel.RBackward compatibility
Code coverage change
https://chatgpt.com/codex/tasks/task_e_68b20598d1b8832691f8251e9349f18d