container tar fix #45
Merged
Conversation
This file contains 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
EthanMarx
added a commit
that referenced
this pull request
Oct 18, 2024
* Law overhaul (#44) * remove old dockerfile and add in apptainer def file * remove old data modules * remove docker ignore * add se config, targets, and parameters * update targets comments * add conda environment * install conda-lock as dev dep * add conda lock * get container build working * add container build workflow * fix up imports and add fetch and segments modules * add authenticate to mldatafind modules * add dev mode * clean up base sandbox tasks * update segment min duration to optional * fix condor imports * debug * get fetch working * clean readme * remove testing workflow * fix up container build workflow] * build sandbox outside of mldatafind dir * build sandbox outside of mldatafind dir * add dummy test * some debug statements * some debug statements * fix path * add law overhal branch for container build * container tar fix (#45) * remove old dockerfile and add in apptainer def file * remove old data modules * remove docker ignore * add se config, targets, and parameters * update targets comments * add conda environment * install conda-lock as dev dep * add conda lock * get container build working * add container build workflow * fix up imports and add fetch and segments modules * add authenticate to mldatafind modules * add dev mode * clean up base sandbox tasks * update segment min duration to optional * fix condor imports * debug * get fetch working * clean readme * remove testing workflow * fix up container build workflow] * build sandbox outside of mldatafind dir * build sandbox outside of mldatafind dir * add dummy test * some debug statements * some debug statements * fix path * add law overhal branch for container build * fix tarring * point to correct app (#46) * fix bug when returning env (#47) * remove sandbox property from data task * Allow python 3.12 * Fix pre-commit * Don't have `dev` option (#49) * check for tmpdir * dont do any dev mounting - should be handled by downstream apps * set /local as string (#50) * Fix bug in tmpdir setting (#51) * set /local as string * fix bug in tmpdir setting * fix bug in tmpdir setting (#52) * Prepare for release (#53) * fix bug in tmpdir setting * use law pypi dep * add pypi publish workflow --------- Co-authored-by: William Benoit <william.benoit@dgx1.ligo-wa.caltech.edu> Co-authored-by: wbenoit26 <90333821+wbenoit26@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.