Skip to content

Fix init issue#97

Merged
transientlunatic merged 2 commits intov0.7-previewfrom
fix-init-issue
Jan 26, 2026
Merged

Fix init issue#97
transientlunatic merged 2 commits intov0.7-previewfrom
fix-init-issue

Conversation

@transientlunatic
Copy link
Collaborator

This PR fixes an issue which casued asimov init Project to fail.

@transientlunatic transientlunatic self-assigned this Jan 26, 2026
@transientlunatic transientlunatic added the bug Something isn't working label Jan 26, 2026
Copilot AI review requested due to automatic review settings January 26, 2026 13:55
Copy link

Copilot AI left a 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 fixes an initialization issue that caused asimov init Project to fail.

Changes:

  • Fixed typo in PR description ("casued" → "caused")
  • Updated test file paths to use "analyses" instead of hardcoded legacy directory names
  • Added comprehensive test coverage for new features including project management, monitor helpers, and optional dependencies

Reviewed changes

Copilot reviewed 122 out of 130 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/test_project.py New test file for Project Python API with context manager support
tests/test_pipelines/test_lalinference.py Updated git add paths to use "analyses" directory
tests/test_pipelines/test_bayeswave.py Updated git add paths and DAG file paths to use "analyses" directory
tests/test_pesummary_subject_analysis.py New test file for PESummary as SubjectAnalysis functionality
tests/test_optional_dependencies.py New test file for optional dependency handling
tests/test_monitor_helpers.py New test file for monitor helper functions
tests/test_monitor_api.py New test file for programmatic monitor API
tests/test_logging.py New test file for logging interface improvements
tests/test_html_report.py New test file for HTML report generation improvements
tests/test_data/*.yaml New test data files for strategy and event testing
tests/test_blueprints/*.yaml New blueprint test files for various pipelines
tests/test_blueprints.py New basic test for blueprints module importability
tests/test_asimov.py Updated to use importlib.metadata instead of pkg_resources
tests/test_application.py Added strategy expansion tests
tests/test_analysis.py Added cleanup to remove tmp directory before test setup
tests/mock_gwdatafind_server.py New mock server for gwdatafind testing
tests/manual_test_priors.py New manual integration test for prior system
tests/integration/test_gwtc2d1.py Updated to use "analyses" directory
tests/integration/GW190426190642.yaml Updated calibration and PSD paths to use "analyses"
tests/test_cli_manage.py Updated file existence checks to use "analyses" directory

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@transientlunatic transientlunatic changed the base branch from main to v0.7-preview January 26, 2026 13:56
@transientlunatic transientlunatic merged commit aa8cff7 into v0.7-preview Jan 26, 2026
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants