[ENH]: Support for Power 2013 coordinates

[synchon]

Are you requiring a new dataset or marker?

• I understand this is not a marker or dataset request

Which feature do you want to include?

We don't yet have the Power 2013 coordinates in-built. This will be a great addition.

How do you imagine this integrated in junifer?

As other in-built coordinates.

Do you have a sample code that implements this outside of junifer?

No response

Anything else to say?

No response

[codecov]

Codecov Report

Merging #245 (fbe18bc) into main (e3e9423) will decrease coverage by 0.06%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
- Coverage   93.10%   93.05%   -0.06%     
==========================================
  Files          84       84              
  Lines        3714     3716       +2     
  Branches      722      723       +1     
==========================================
  Hits         3458     3458              
- Misses        160      161       +1     
- Partials       96       97       +1     

Flag	Coverage Δ
docs	100.00% <ø> (ø)
junifer	93.05% <33.33%> (-0.06%)	⬇️

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

Files Changed	Coverage Δ
junifer/data/coordinates.py	95.34% <33.33%> (-4.66%)	⬇️

[github-actions]

PR Preview Action v1.4.4
🚀 Deployed preview to https://juaml.github.io/junifer/pr-preview/pr-245/
on branch gh-pages at 2023-09-06 09:39 UTC

[fraimondo]

This is a backward's incompatible change. What about a deprecation cycle?

If I remember correctly, we can use functions to "load" coordinates/masks. Or we can implement it in a way that when someone uses "Power" it triggers a deprectation warning during a few release cycles. (We need to define).

[synchon]

This is a backward's incompatible change. What about a deprecation cycle?

That is true. I went with the breaking change as we are still not v1.0.0 . But, I don't mind putting a deprecation notice.

If I remember correctly, we can use functions to "load" coordinates/masks. Or we can implement it in a way that when someone uses "Power" it triggers a deprectation warning during a few release cycles. (We need to define).

So, I'll put a deprecation notice stating that we'll remove it in the next release.