Skip to content

Add HTCondor file transfer#446

Draft
eagoetz wants to merge 1 commit into
gwpy:masterfrom
eagoetz:batch-file-transfer
Draft

Add HTCondor file transfer#446
eagoetz wants to merge 1 commit into
gwpy:masterfrom
eagoetz:batch-file-transfer

Conversation

@eagoetz

@eagoetz eagoetz commented May 21, 2026

Copy link
Copy Markdown
Collaborator

This PR adds the HTCondor file transfer to vanilla universe jobs.

I had to unify the argument parsing for the gw_summary and gw_summary_pipe entry points because I needed the output path to the directory where the gw_summary program is outputting files.

@eagoetz eagoetz added this to the 2.5.0 milestone May 21, 2026
@eagoetz eagoetz self-assigned this May 21, 2026
@eagoetz
eagoetz force-pushed the batch-file-transfer branch 8 times, most recently from 646e3f9 to d9b4eeb Compare May 23, 2026 00:26
@codecov

codecov Bot commented May 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.77778% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.84%. Comparing base (78d480b) to head (094cdfe).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
gwsumm/batch.py 88.37% 10 Missing ⚠️
gwsumm/utils.py 97.10% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #446      +/-   ##
==========================================
+ Coverage   49.26%   49.84%   +0.58%     
==========================================
  Files          60       60              
  Lines        8806     8942     +136     
==========================================
+ Hits         4338     4457     +119     
- Misses       4468     4485      +17     
Flag Coverage Δ
Linux-python3.10 49.84% <93.78%> (+0.58%) ⬆️
Linux-python3.11 49.82% <93.78%> (+0.58%) ⬆️
macOS-python3.10 49.84% <93.78%> (+0.58%) ⬆️
macOS-python3.11 49.82% <93.78%> (+0.58%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eagoetz
eagoetz force-pushed the batch-file-transfer branch 14 times, most recently from 9b5f7cd to acf67c5 Compare May 29, 2026 01:08
@eagoetz
eagoetz force-pushed the batch-file-transfer branch from acf67c5 to 54f6ae3 Compare June 1, 2026 19:57
@eagoetz
eagoetz force-pushed the batch-file-transfer branch 4 times, most recently from 3e3f2a1 to 4c36cb6 Compare June 19, 2026 00:17
@eagoetz
eagoetz force-pushed the batch-file-transfer branch from 4c36cb6 to 6c90e77 Compare June 22, 2026 21:14
@eagoetz
eagoetz force-pushed the batch-file-transfer branch from 6c90e77 to 094cdfe Compare June 22, 2026 21:36
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.

1 participant