Skip to content

Changed pint.ls to astropy.u.lsec#2008

Merged
abhisrkckl merged 5 commits into
nanograv:masterfrom
dlakaplan:removels
Jun 16, 2026
Merged

Changed pint.ls to astropy.u.lsec#2008
abhisrkckl merged 5 commits into
nanograv:masterfrom
dlakaplan:removels

Conversation

@dlakaplan

Copy link
Copy Markdown
Contributor

Astropy added a light-second unit years ago. I think rather than defining our own (which used astropy's definition of c) we should use their version throughout.

This might break some downstream software, so if needed we could put back the definition of pint.ls = u.lsec, although we do use the pint_units as keys for string parsing which includes ls.

@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.39%. Comparing base (f5dfaf6) to head (f3d877a).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2008      +/-   ##
==========================================
+ Coverage   70.32%   70.39%   +0.07%     
==========================================
  Files         109      109              
  Lines       25745    25743       -2     
  Branches     4106     4107       +1     
==========================================
+ Hits        18105    18122      +17     
+ Misses       6473     6450      -23     
- Partials     1167     1171       +4     

☔ 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.

@dlakaplan dlakaplan added awaiting review This PR needs someone to review it so it can be merged minor A minor PR that doesn't need a lot of thought labels Jun 11, 2026
@abhisrkckl

Copy link
Copy Markdown
Contributor

This looks good. I will merge this if ready.

@dlakaplan

Copy link
Copy Markdown
Contributor Author

This looks good. I will merge this if ready.

Yes, once the latest merge finishes.

@abhisrkckl abhisrkckl merged commit bc27ca0 into nanograv:master Jun 16, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review This PR needs someone to review it so it can be merged minor A minor PR that doesn't need a lot of thought

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants