Skip to content

Conversation

molinaro-m
Copy link
Member

This is a first, baseline, attempt at putting Cone Search registration prescription as a direct reference to SimpleDALRegExt.

The final goal is to close issue #62.

Please comment and discuss here what else should be part of this change in the ConeSearch text, taking into account that an Erratum will be required for SimpleDALRegExt to smooth out minor details.

For example, should we explicitly note here that there's an Endorsed Note (XML Schema Versioning) that saves us from changing standardID and namespace?

I suggest to keep this pull request in draft for some time to allow all interested people to be aware of this change.

msdemlei
msdemlei previously approved these changes Sep 19, 2025
Copy link
Collaborator

@msdemlei msdemlei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I wonder if some advisory text on how to discover SCS services would be in order ("where standard_id like 'ivo://ivoa.net/std/conesearch%'"). Also, we could be specific on what to do with capability/interface/version; right now, most services have this empty. whereas others put in 1.03, 1.1, or 1.0. This is not dramatically important, but it might be convenient if someone (e.g., a validator author) actually wants this. See:

select distinct std_version from rr.interface
natural join rr.capability
where standard_id like 'ivo://ivoa.net/std/conesearch%'

But all that can wait. Let's merge this and see.

@molinaro-m
Copy link
Member Author

Just to add some text to the section, maybe we can add the ADQL query on standardID as a showcase.
It would be a couple of text lines and the query itself, copying your comment here above.

Should I try it?

The part on version might, on the other hand, become a new issue for v1.1 to be discussed.

@msdemlei
Copy link
Collaborator

msdemlei commented Sep 19, 2025 via email

@molinaro-m
Copy link
Member Author

I added a few lines related to service discovery via RegTAP.
I also tried out a sentence on std_version potential usage.
Please, @msdemlei , could you re-review this request?

@molinaro-m molinaro-m marked this pull request as ready for review September 22, 2025 09:53
msdemlei
msdemlei previously approved these changes Sep 22, 2025
Copy link
Collaborator

@msdemlei msdemlei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the discovery query, add a "AND intf_role='std'"; people shouldn't add other interfaces to cone search capabilities, but better safe than sorry (cf. https://ivoa.net/documents/RegTAP/20241002/REC-RegTAP-1.2.html#tth_sEc10.1).

I'll approve, and feel free to merge once you've put it in. Thanks!

@molinaro-m
Copy link
Member Author

Fixed, thanks!
Unfortunately I cannot merge my own PR.
Whenever you or someone else approves, I'll merge and close the related issue.

Copy link
Collaborator

@msdemlei msdemlei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@msdemlei msdemlei merged commit 2688499 into ivoa-std:main Sep 22, 2025
1 check passed
@msdemlei msdemlei deleted the reg-ext branch September 22, 2025 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants