Merge SkyCoord support to Master

cds-astro / ipyaladin

An IPython Widget for Aladin Lite, the sky viewer.

https://cds-astro.github.io/ipyaladin/

BSD 3-Clause "New" or "Revised" License

126 stars 26 forks source link

Closed Xen0Xys closed 5 months ago

Xen0Xys commented 6 months ago

[x] Fix positionChanged event when position.ra/dec is different that getRaDec result
[x] Fix example 6 bug with position sync between multiple aladin views
[x] Add SkyCoord when the target is a coordinate string
[x] Add documentation
[x] Check that existing examples are always working
[x] Fix target initialization parameter
[x] Add testing for parsing methods
[x] Add changelog

ManonMarchand commented 6 months ago

Thanks Tom!

Fixes: #51

For the gitignore, you can edit your local .git/info/exclude to avoid editing the shared one

I'll have a look at the rest tomorrow :slightly_smiling_face:

Edit: could you rebase on master? I tried to fix the codestyle workflow

ManonMarchand commented 5 months ago

Merged, thanks Tom! :tada: