Conversation
📊 Test Coverage ReportStatus: ❌ Coverage report not available Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
1 similar comment
📊 Test Coverage ReportStatus: ❌ Coverage report not available Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
1 similar comment
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
e6719c5 to
61bbc85
Compare
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
2 similar comments
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 45 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 45 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 45 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 45 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📊 Test Coverage ReportStatus: ✅ PASS (Coverage: 100.00% | Threshold: 100%)
Coverage report generated by pytest-cov. Full HTML report available in workflow artifacts. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 45 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 45 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "test-local": { | ||
| "provider": "localhost", | ||
| "cores_per_node": 1, | ||
| "max_workers_per_node": 1, | ||
| "cores_per_node": 2, | ||
| "max_workers_per_node": 2, | ||
| "environment": [f"export PYTHONPATH=${{PYTHONPATH}}:{project_root}"], |
There was a problem hiding this comment.
The test configuration hard-codes 2 cores/workers for the localhost provider. This can make the test suite fail or behave unexpectedly on constrained runners (e.g., single-core CI/dev containers). Consider deriving these values from os.cpu_count() (e.g., min(2, cpu_count)), or keep the default at 1 and only request 2 workers in the specific test that needs concurrency.
| @python_task | ||
| def transfer_task( | ||
| src_ep: str, | ||
| dst_ep: str, | ||
| src_path: str, | ||
| dst_path: str, | ||
| timeout: int = 3600, | ||
| polling_interval: int = 30, | ||
| client: Optional[TransferClient] = None, | ||
| recursive: bool = False, | ||
| dependencies: Optional[Future] = None, | ||
| ): |
There was a problem hiding this comment.
transfer_task and delete_task removed the previously documented dependencies parameter. If this was part of the public API, callers will now get a TypeError when passing dependencies=. Consider keeping dependencies as a deprecated alias that maps to Parsl's inputs (or at least raising a clearer error message guiding users to inputs=[...]).
This updates the documentation to a complete first draft. It also updates the licensing for the repository and provides an initial Conda recipe for uploading to conda-forge. Some API changes were also made when creating the documentation exposed some problems. Closes #111