Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move Unified Content Descriptors to UCD1+ standard #20

Closed
molinaro-m opened this issue Jul 24, 2020 · 5 comments
Closed

Move Unified Content Descriptors to UCD1+ standard #20

molinaro-m opened this issue Jul 24, 2020 · 5 comments
Labels

Comments

@molinaro-m
Copy link
Member

Although this is definitely a major change in the specification, we should try to move the UCDs listed in the ConeSearch protocol to the UCD1+ syntax.

The following UCD 1.0 words are the only ones used in ConeSearch 1.03:

  • ID_MAIN
  • POS_EQ_RA_MAIN
  • POS_EQ_DEC_MAIN
  • OBS_ANG-SIZE
molinaro-m added a commit to molinaro-m/ConeSearch that referenced this issue Jul 24, 2020
@molinaro-m
Copy link
Member Author

A direct move to UCD1+ breaks back-compatibility, on the theoretical level.
In practice, being able to do so for the above 4 UCDs, a lot depends on the ability of client applications to deal with either one UCD format or server/client communication about the supported protocol version.

@molinaro-m molinaro-m added critical question Further information is requested labels Aug 27, 2020
@mbtaylor
Copy link
Member

mbtaylor commented Sep 2, 2020

(I am reasonably sure that) TOPCAT/STILTS will not have a problem with cone search services that return UCD1+ instead of UCD1.

@molinaro-m
Copy link
Member Author

An investigation on the usage of the UCD / UCD1+ couples in registered ConeSearches

  • ID_MAIN / meta.id;meta.main
  • POS_EQ_RA_MAIN / pos.eq.ra;meta.main
  • POS_EQ_DEC_MAIN / pos.eq.dec;emat.main
  • OBS_ANG-SIZE / phys.angSize;obs

shows that there are few cases where the switch to the current UCD Recommendation could lead to issues on the data providers side and, in any case, that those issues are related to the meaning of the "main identifier" for the record in the catalogue, not the update of the UCD specification.
(A few more details are available in the notes from this DAL running meeting)

This "main ID" issue will be taken care by a specific repo issue: #53 (main ID usage).

This issue will continue to stay open to further investigate (mainly) client side concerns.

@molinaro-m
Copy link
Member Author

Pull Request #58 is supposed to fix and close this issue, pending #55 merge in advance

@molinaro-m molinaro-m added 1.1 and removed question Further information is requested critical labels Nov 8, 2023
@molinaro-m
Copy link
Member Author

closed via #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants