-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added URL posts to register CLEM workflow files #357
base: main
Are you sure you want to change the base?
Conversation
tieneupin
commented
Oct 2, 2024
- Additionally, fixed minor bugs found in code after last minor update.
… this Murfey workflow
…ed LIF file registration and processing parameters; corrected 'murfey_workflow' call
…FF files; packaged posts as functions
def register_tiff_file( | ||
self, | ||
tiff_file: Path, | ||
environment: MurfeyInstanceEnvironment, | ||
): |
Check notice
Code scanning / CodeQL
Explicit returns mixed with implicit (fall through) returns Note
def process_tiff_series( | ||
self, | ||
tiff_dataset: dict, | ||
environment: MurfeyInstanceEnvironment, | ||
): |
Check notice
Code scanning / CodeQL
Explicit returns mixed with implicit (fall through) returns Note
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #357 +/- ##
==========================================
- Coverage 29.12% 28.98% -0.14%
==========================================
Files 70 70
Lines 9481 9540 +59
Branches 1747 1756 +9
==========================================
+ Hits 2761 2765 +4
- Misses 6618 6673 +55
Partials 102 102 |
Validation checks:
The cluster submission ones are too challenging for me to test at present, but both registration functions work when I construct the URL and post it via |
…ed additional 'raise Exception' to catch unexpected ones when loading rsync_basepath