Merge SkyCoord support to Master

.github/workflows/python-tests.yml

@@ -0,0 +1,25 @@
+name: python-tests
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+  # Allows to run this workflow manually from the Actions tab
+  workflow_dispatch:
+jobs:
+  python-tests:
+    runs-on: ubuntu-latest
+    steps:
+      - name: "Checkout branch"
+        uses: actions/checkout@v4
+      - name: "Set up Python on Ubuntu"
+        uses: actions/setup-python@v5
+        with:
+          python-version: 3.12
+      - name: "Python codestyle"
+        run: |
+          pip install ".[dev]"
+          pip install pytest
+          pytest .

CHANGELOG.md

@@ -14,6 +14,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Fixed** for any bug fixes.
 - **Security** in case of vulnerabilities.
 
+## [Unreleased]
+
+### Added
+
+- Support for `astropy.coordinates.SkyCoord` for the `target` property (#80)
+
+### Fixed
+
+- Fix asynchronous update for the `target` property (#80)
+
+### Changed
+
+- Change the jslink target trait from `target` to `shared_target` (#80)
+
 ## [0.3.0]
 
 ### Changed

examples/6_Linked-widgets.ipynb

@@ -27,8 +27,8 @@
     "c = Aladin(layout=Layout(width=\"33.33%\"), survey=\"P/2MASS/color\", **cosmetic_options)\n",
     "\n",
     "# synchronize target between 3 widgets\n",
-    "widgets.jslink((a, \"target\"), (b, \"target\"))\n",
-    "widgets.jslink((b, \"target\"), (c, \"target\"))\n",
+    "widgets.jslink((a, \"shared_target\"), (b, \"shared_target\"))\n",
+    "widgets.jslink((b, \"shared_target\"), (c, \"shared_target\"))\n",
     "\n",
     "# synchronize FoV (zoom level) between 3 widgets\n",
     "widgets.jslink((a, \"fov\"), (b, \"fov\"))\n",

js/widget.js

@@ -4,6 +4,7 @@ import "./widget.css";
 let idxView = 0;
 
 function convert_pyname_to_jsname(pyname) {
+  if (pyname.charAt(0) === "_") pyname = pyname.slice(1);
   let temp = pyname.split("_");
   for (let i = 1; i < temp.length; i++) {
     temp[i] = temp[i].charAt(0).toUpperCase() + temp[i].slice(1);
@@ -34,6 +35,9 @@ function render({ model, el }) {
   let aladin = new A.aladin(aladinDiv, init_options);
   idxView += 1;
 
+  const ra_dec = init_options["target"].split(" ");
+  aladin.gotoRaDec(ra_dec[0], ra_dec[1]);
+
   el.appendChild(aladinDiv);
 
   /* ------------------- */
@@ -46,27 +50,34 @@ function render({ model, el }) {
   // the gotoObject call should only happen once. The two booleans prevent the two
   // listeners from triggering each other and creating a buggy loop. The same trick
   // is also necessary for the field of view.
+
+  /* Target control */
   let target_js = false;
   let target_py = false;
 
-  aladin.on("positionChanged", (position) => {
-    if (!target_py) {
-      target_js = true;
-      model.set("target", `${position.ra} ${position.dec}`);
-      model.save_changes();
-    } else {
+  // Event triggered when the user moves the map in Aladin Lite
+  aladin.on("positionChanged", () => {
+    if (target_py) {
       target_py = false;
+      return;
     }
+    target_js = true;
+    const ra_dec = aladin.getRaDec();
+    model.set("_target", `${ra_dec[0]} ${ra_dec[1]}`);
+    model.set("shared_target", `${ra_dec[0]} ${ra_dec[1]}`);
+    model.save_changes();
   });
 
-  model.on("change:target", () => {
-    if (!target_js) {
-      target_py = true;
-      let target = model.get("target");
-      aladin.gotoObject(target);
-    } else {
+  // Event triggered when the target is changed from the Python side using jslink
+  model.on("change:shared_target", () => {
+    if (target_js) {
       target_js = false;
+      return;
     }
+    target_py = true;
+    const target = model.get("shared_target");
+    const [ra, dec] = target.split(" ");
+    aladin.gotoRaDec(ra, dec);
   });
 
   /* Field of View control */
@@ -182,6 +193,11 @@ function render({ model, el }) {
   model.on("msg:custom", (msg) => {
     let options = {};
     switch (msg["event_name"]) {
+      case "goto_ra_dec":
+        const ra = msg["ra"];
+        const dec = msg["dec"];
+        aladin.gotoRaDec(ra, dec);
+        break;
       case "add_catalog_from_URL":
         aladin.addCatalog(A.catalogFromURL(msg["votable_URL"], msg["options"]));
         break;
@@ -235,7 +251,6 @@ function render({ model, el }) {
 
   return () => {
     // need to unsubscribe the listeners
-    model.off("change:target");
     model.off("change:fov");
     model.off("change:height");
     model.off("change:coo_frame");

pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "hatchling.build"
 [project]
 name = "ipyaladin"
 dynamic = ["version"]
-dependencies = ["anywidget"]
+dependencies = ["anywidget", "astropy"]
 readme = "README.md"
 
 [project.optional-dependencies]

src/ipyaladin/__init__.py

@@ -1,9 +1,12 @@
 import importlib.metadata
 import pathlib
-from typing import ClassVar
 import warnings
 
+from astropy.coordinates import SkyCoord
+
 import anywidget
+
+from typing import ClassVar
 from traitlets import (
     Float,
     Int,
@@ -16,6 +19,7 @@
     default,
     Undefined,
 )
+from .coordinate_parser import parse_coordinate_string
 
 try:
     __version__ = importlib.metadata.version("ipyaladin")
@@ -29,7 +33,16 @@ class Aladin(anywidget.AnyWidget):
 
     # Options for the view initialization
     height = Int(400).tag(sync=True, init_option=True)
-    target = Unicode("0 0").tag(sync=True, init_option=True)
+    _target = Unicode(
+        "0 0",
+        help="this trait is used belong the target property "
+        "to store the current target of Aladin Lite",
+    ).tag(sync=True, init_option=True)
+    shared_target = Unicode(
+        "0 0",
+        help="this trait is destined to be used with jslink widget function "
+        "to link two Aladin Lite widgets target together",
+    ).tag(sync=True)
     fov = Float(60.0).tag(sync=True, init_option=True)
     survey = Unicode("https://alaskybis.unistra.fr/DSS/DSSColor").tag(
         sync=True, init_option=True
@@ -85,6 +98,7 @@ def _init_options(self):
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
+        self.target = kwargs.get("target", "0 0")
         self.on_msg(self._handle_custom_message)
 
     def _handle_custom_message(self, model, message, list_of_buffers):  # noqa: ARG002
@@ -105,6 +119,51 @@ def _handle_custom_message(self, model, message, list_of_buffers):  # noqa: ARG0
         elif event_type == "select" and "select" in self.listener_callback:
             self.listener_callback["select"](message_content)
 
+    @property
+    def target(self) -> SkyCoord:
+        """
+        Get the target of the Aladin Lite widget.
+        :return: astropy.coordinates.SkyCoord object
+        """
+        ra, dec = self._target.split(" ")
+        return SkyCoord(
+            ra=ra,
+            dec=dec,
+            frame="icrs",
+            unit="deg",
+        )
+
+    @target.setter
+    def target(self, target: str or SkyCoord):
+        """
+        Set the target of the Aladin Lite widget.
+        :param target: string or astropy.coordinates.SkyCoord object
+        :return: None
+        """
+        if isinstance(target, str):  # If the target is a string, parse it
+            sc = parse_coordinate_string(target)
+            self._target = f"{sc.icrs.ra.deg} {sc.icrs.dec.deg}"
+            self.send(
+                {
+                    "event_name": "goto_ra_dec",
+                    "ra": sc.icrs.ra.deg,
+                    "dec": sc.icrs.dec.deg,
+                }
+            )
+        elif isinstance(target, SkyCoord):  # If the target is a SkyCoord object
+            self._target = f"{target.icrs.ra.deg} {target.icrs.dec.deg}"
+            self.send(
+                {
+                    "event_name": "goto_ra_dec",
+                    "ra": target.icrs.ra.deg,
+                    "dec": target.icrs.dec.deg,
+                }
+            )
+        else:
+            raise ValueError(
+                "target must be a string or an astropy.coordinates.SkyCoord object"
+            )
+
     def add_catalog_from_URL(self, votable_URL, votable_options=None):
         """load a VOTable table from an url and load its data into the widget
         Args:

src/ipyaladin/coordinate_parser.py

@@ -0,0 +1,31 @@
+from astropy.coordinates import SkyCoord, Angle
+import re
+
+
+def parse_coordinate_string(string: str) -> SkyCoord:
+    regex = r"^[0-9][0-9: hmsd.+-]+$"
+    if not re.match(regex, string):
+        return SkyCoord.from_name(string)
+    coordinates: tuple[str, str] = _split_coordinate_string(string)
+    ra: Angle or None = None
+    dec: Angle = Angle(coordinates[1], unit="deg")
+    if _is_hour_angle_string(coordinates[0]):
+        ra = Angle(coordinates[0], unit="hour")
+    else:
+        ra = Angle(coordinates[0], unit="deg")
+    return SkyCoord(ra=ra, dec=dec, frame="icrs")
+
+
+def _split_coordinate_string(coo: str) -> tuple[str, str]:
+    regex = r"[\s°]"
+    parts = re.split(regex, coo)
+    parts = [part for part in parts if part]
+    middle = len(parts) // 2
+    first_part = " ".join(parts[:middle])
+    second_part = " ".join(parts[middle:])
+    return first_part, second_part
+
+
+def _is_hour_angle_string(coo: str) -> bool:
+    regex = r"[hms°: ]"
+    return bool(re.search(regex, coo))