Skip to content

Update the observation date modeling that has been changed during the#726

Open
lmichel wants to merge 6 commits intoastropy:mainfrom
lmichel:fix-mango-obsdate
Open

Update the observation date modeling that has been changed during the#726
lmichel wants to merge 6 commits intoastropy:mainfrom
lmichel:fix-mango-obsdate

Conversation

@lmichel
Copy link
Contributor

@lmichel lmichel commented Feb 4, 2026

During the MANGO REC process, it has been suggested to simplify the representation of the observation dates (mango:EpochPostion.obsDate).
The mango:ObsDate complex type has been replaced with a simple attribute where the representation is given by its dmtype (<ATTRIBUTE dmrole="mango:EpochPosition.obsDate" dmtype="year" value="2000.0" \>).

This change requires the SkyCoord builder to be updated as well as the annotation builder.

MANGO REC process. This is now a simple attribute instead of a complex
object.
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.15%. Comparing base (8316767) to head (f89b913).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #726      +/-   ##
==========================================
+ Coverage   84.10%   84.15%   +0.04%     
==========================================
  Files          79       79              
  Lines        8608     8603       -5     
==========================================
  Hits         7240     7240              
+ Misses       1368     1363       -5     

☔ View full report in Codecov by Sentry.
📢 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.

@bsipocz bsipocz added this to the v1.9 milestone Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants