Skip to content

Update segments module to handle incoming changes from gwpy DataQualityDict updates - #417

Open
eagoetz wants to merge 1 commit into
gwpy:masterfrom
eagoetz:rename-url-host
Open

eagoetz wants to merge 1 commit into
gwpy:masterfrom
eagoetz:rename-url-host

Conversation

@eagoetz

@eagoetz eagoetz commented Dec 5, 2024 •

Copy link
Copy Markdown
Collaborator

This PR updates the segments module to handle changes to the gwpy.DataQualityDict API changing url keyword to host.

Also some minor clean up of the code and setting new version pins.

@eagoetz eagoetz self-assigned this Dec 5, 2024
@eagoetz eagoetz added this to the 2.3.0 milestone Dec 5, 2024
@eagoetz
eagoetz force-pushed the rename-url-host branch 3 times, most recently from 6380af5 to 332d620 Compare December 13, 2024 18:49
@codecov

codecov Bot commented Dec 13, 2024 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 55.00000% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.24%. Comparing base (51a9653) to head (c36b9fe).

Files with missing lines Patch % Lines
gwsumm/segments.py 55.56% 8 Missing ⚠️
gwsumm/state/core.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #417      +/-   ##
==========================================
- Coverage   49.25%   49.24%   -0.01%     
==========================================
  Files          60       60              
  Lines        8811     8806       -5     
==========================================
- Hits         4339     4336       -3     
+ Misses       4472     4470       -2     
Flag Coverage Δ
Linux-python3.10 49.24% <55.00%> (-0.01%) ⬇️
Linux-python3.11 49.22% <55.00%> (-0.01%) ⬇️
Linux-python3.12 49.22% <55.00%> (-0.01%) ⬇️
Linux-python3.13 49.22% <55.00%> (-0.01%) ⬇️
macOS-python3.10 49.24% <55.00%> (-0.01%) ⬇️
macOS-python3.11 49.22% <55.00%> (-0.01%) ⬇️
macOS-python3.12 49.22% <55.00%> (-0.01%) ⬇️
macOS-python3.13 49.22% <55.00%> (-0.01%) ⬇️

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 modified the milestones: 2.3.0, 2.3.1 Apr 28, 2025
@eagoetz eagoetz modified the milestones: 2.3.1, 2.4.0 May 7, 2025
@eagoetz eagoetz modified the milestones: 2.4.0, 2.4.1 Nov 25, 2025
@eagoetz
eagoetz force-pushed the rename-url-host branch 3 times, most recently from 5edd0f7 to c3b0eaa Compare May 14, 2026 23:00
@eagoetz
eagoetz marked this pull request as ready for review May 14, 2026 23:54
@eagoetz eagoetz modified the milestones: 2.4.1, 2.5.0 May 15, 2026
@eagoetz
eagoetz force-pushed the rename-url-host branch 3 times, most recently from 088d2f4 to 69a12cf Compare September 28, 2026 18:26
@eagoetz
eagoetz requested review from areeda and removed request for duncanmmacleod and iaraota September 28, 2026 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant