Skip to content

Trim environments and containers - #521

Merged
wbenoit26 merged 6 commits into
ML4GW:devfrom
wbenoit26:slim-envs
Sep 22, 2026
Merged

wbenoit26 merged 6 commits into
ML4GW:devfrom
wbenoit26:slim-envs

Conversation

@wbenoit26

@wbenoit26 wbenoit26 commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Built on #519 and will be rebased once it's merged.

  • Ensures that every project/lib declares any dependency it imports, and doesn't declare anything unused.
  • Purges build tools from containers to keep them smaller
  • Use a uv workspace to unify all the uv.locks into a single lockfile

I'll be performing an end-to-end test to make sure everything still runs.

@wbenoit26 wbenoit26 mentioned this pull request Sep 21, 2026
Comment thread docs/first_pipeline.md
2. Generate testing data
3. Train or Tune a model
3. Train a model
4. Export trained weights to TensorRT

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This won't be limited to just TensorRT and Triton, right? This could also include the in-process workflow?

Comment thread docs/first_pipeline.md
.. note::
Running the sandbox pipeline out-of-the-box requires access to an enterprise-grade GPU(s) (e.g. P100, V100, T4, A[30,40,100], etc.). There are several nodes on the LIGO Data Grid which meet these requirements_**.
.. note::
Running the pipeline out-of-the-box requires access to an enterprise-grade GPU(s) (e.g. P100, V100, T4, A[30,40,100], etc.). There are several nodes on the LIGO Data Grid which meet these requirements.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My above comment is answered here I believe

Comment thread pipeline/config/config.yaml

@bhgupta20 bhgupta20 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! I'd a few comments but they got answered while reviewing the PR.

@wbenoit26
wbenoit26 merged commit 76fcb80 into ML4GW:dev Sep 22, 2026
15 checks passed
@wbenoit26
wbenoit26 deleted the slim-envs branch September 22, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants