Upgrade to python 3 #50
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
HealthyPear
added
documentation
Codecov Report
@@ Coverage Diff @@
## master #50 +/- ##
==========================================
- Coverage 21.82% 21.61% -0.21%
==========================================
Files 10 10
Lines 747 745 -2
==========================================
- Hits 163 161 -2
Misses 584 584
Continue to review full report at Codecov.
|
Closed
- move all material to a dedicated folder - store here conde environment files, which are now also updated
- directory tree test is skipped until protopipe 0.5.0 is available - CI has been updated to only support conda-based installation also on macos - yaml needed to be added to the test dependencies - conda recipes moved back to root folder Update CI Add pyyaml as a dependency for testing Update CI installation Remove unused import Fix CI installation order Test CI on macos-latest Try giving CONDA permission to its own files Update CI - add macos-latest - update conda env name Forgot to unset env name Use unambiguous conda env name Import load_config from utils not from protopipe also skip test that requires new protopipe leave protopipe import in submit jobs Test CI only on macos-latest Test CI only with conda-based installation Test CI also on ubuntu-latest but only conda
HealthyPear
force-pushed
the
upgrade_python_3
branch
from
0b84bbc
to
0a6dfc5
Compare
- added an option to disable the upload of the analysis config file - added a loop to counteract some Dirac issues when splitting input data
This was referenced Jan 21, 2022
Merged
- Output type was showing input sandbox - added protopipe version used at submission
- strip newlines trailing characters from list entries, - use logging.exception in case list is not found, - use absolute file path from root directory
- for some reason now fts3 needs to be specified - python version also specified as in Dirac and CTADIRAC Update conda envs
HealthyPear
force-pushed
the
upgrade_python_3
branch
from
53231f9
to
a3a4293
Compare
Merged
2 tasks
This was referenced Feb 15, 2022
kosack
approved these changes
Feb 16, 2022
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.
Summary
pipwithin acondaenvironmentcontainers; we should at least support a Docker container