-
Notifications
You must be signed in to change notification settings - Fork 10
feat(fusion): parameter sweep for doctor validation #44
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
Open
adamrtalbot
wants to merge
55
commits into
main
Choose a base branch
from
COMP-POC/fusion-doctor-parameter-sweep
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 51 commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
b19e7be
feat(fusion-doctor): Add Python report generator and HTML template
alberto-miranda 05b3ff7
test(fusion-doctor): Add unit tests for report generation
alberto-miranda ab418fb
feat(fusion-doctor): Add `FUSION_DOCTOR_GENERATE_REPORT` process
alberto-miranda b150c34
chore: Update test snapshots
alberto-miranda 02a8dad
ci: Integrate Python tests in CI pipelines
alberto-miranda fc6768e
fix: Make `fusion_report_template.html` conformant with `prettier`
alberto-miranda 33ea648
fix: Add `uv` container for `FUSION_DOCTOR_GENERATE_REPORT`
alberto-miranda 8d0d96d
fix: Replace `uv:0.10.6` container with `uv:python3.12-bookworm-slim`
alberto-miranda e814e44
ci: Add `setup-uv` to `nf-test` job
alberto-miranda 53afe7e
fix: Replace `uv:python3.12-bookworm-slim` with `jinja2_python_uv:711…
alberto-miranda 047347b
chore: Rename HTML report to `fusion-report.html`
alberto-miranda 09220c1
feat(fusion-doctor): Split profiles into tiers
alberto-miranda 676f39c
fix: Stage report template file as an input
alberto-miranda 6b77f7f
fix: Ensure `uv` does not check for project files
alberto-miranda 5e8a31e
fix: Pass template file as parameter instead of copying to fixed loca…
alberto-miranda f3a7b72
fix: Handle real fusion doctor JSON schema in HTML report
alberto-miranda d581249
fix: Fix report generation
alberto-miranda 9ee58b6
feat: Enhance Fusion diagnostic report with system, storage, and reso…
alberto-miranda f045e99
feat: Redesign fusion report with Platform design system and polish
alberto-miranda 0e37e90
fix: Add humanized labels for new fusion doctor checks
alberto-miranda a173a18
feat: Add nvme, cpu_cores, and open_files checks to fusion profiles
alberto-miranda 0398577
feat: validate single Fusion profile usage for TEST_FUSION_DOCTOR
alberto-miranda 6b74eb6
fix: Update report names in tests
alberto-miranda c9af456
feat: Dark header matching Seqera Platform navigation bar
alberto-miranda aae734a
feat: Color-coded overview cards and status badge in overview grid
alberto-miranda a359e5c
feat: Convert validation checks from cards to table with collapsible …
alberto-miranda e99b57a
fix: Design critique polish for v4 report template
alberto-miranda c785947
feat: Allow TEST_FUSION_DOCTOR to continue with exit codes 1 and 3
alberto-miranda 377a1e8
feat: Redesign fusion report with Platform design system
alberto-miranda 6ab31b1
feat: Improve accessibility and template structure
alberto-miranda b3d77bb
feat: Add mobile responsive layout
alberto-miranda 2d1423e
feat: Split bucket table into access + URI columns
alberto-miranda 01f4a27
feat: Add footer and improve visual polish
alberto-miranda 0fa6260
refactor: Simplify, deduplicate, and remove dead code
alberto-miranda 2db3600
test: Improve Python tests
alberto-miranda d00005d
fix: Exclude Jinja2 templates from Prettier checks
alberto-miranda 93145be
fix: Run Python tests via pytest in CI to fix module import
alberto-miranda b41373c
feat(fusion): Allow TEST_FUSION_DOCTOR to proceed on validation failures
alberto-miranda 9f08932
fix(fusion): Improve report layout and resource limits display
alberto-miranda 26a60ea
fix(fusion): Lower open_files_min to 65535 in low-tier profiles
alberto-miranda acfd727
feat(fusion): Add example doctor report fixtures and generator script
alberto-miranda bffe20e
refactor(fusion): Remove duplicate details from resource limits table
alberto-miranda 90b8b8c
feat(fusion): Add `fusion_redact` param to mask PII in diagnostics
alberto-miranda 0766a8e
chore: Leave profile handling to Nextflow
alberto-miranda 18f2a2d
chore: Replace horizontal dividers with subheadings
alberto-miranda f887634
fix: Clarify NVMe auto-selection for AWS CEs
alberto-miranda 01ccfe5
fix: Make path mandatory in `load_json_report`
alberto-miranda 0e7735d
refactor(tests): Convert fusion doctor report test to process-level
alberto-miranda 66e90f8
chore: Remove example report fixtures
alberto-miranda 8c38d40
refactor: Remove try/catch from `load_json_report`, catch at `main()`…
alberto-miranda 061ccc3
feat(fusion): add parameter sweep for fusion doctor validation
adamrtalbot 1e4a587
Apply suggestions from code review
adamrtalbot 5e234d4
Merge origin/main: integrate dedicated fusion-doctor container and HT…
Copilot 3116670
chore: initial plan for merge conflict resolution
Copilot 996f576
fix: resolve merge conflicts with origin/main
Copilot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,4 +13,5 @@ results/ | |
| *.test | ||
| *.test.snap | ||
| *.txt | ||
| assets/templates/ | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.