Skip to content

Adding a StandardsRegExt record for VOEvent. #3

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

Merged
merged 4 commits into from
May 22, 2020

Conversation

msdemlei
Copy link
Contributor

@msdemlei msdemlei commented May 6, 2020

This ought to be mailed to the RofR as soon as we're happy with it, and
then updated when a new version is issued.

I've taken the liberty to include a few standard keys required in
the proposal for registering VOEvent streams that'll come in a separate
PR. They don't hurt if if that's rejected (but probably should still
be taken out if that happens).

msdemlei added 2 commits May 6, 2020 09:58
This ought to be mailed to the RofR as soon as we're happy with it, and
then updated when a new version is issued.

I've taken the liberty to include a few standard keys required in
the proposal for registering VOEvent streams that'll come in a separate
PR.  They don't hurt if if that's rejected (but probably should still
be taken out if that happens).
This commit also adds a "proprietary" access protocol as a catch-all option.

<endorsedVersion status="rec">2.0</endorsedVersion>

<key>
Copy link
Member

Choose a reason for hiding this comment

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

I'm not convinced this should be a fixed list. Is there a way to refer to the transport standards using a list of URIs ?

<curation>
<publisher>IVOA</publisher>

<creator><name>Seaman, Rob</name></creator>
Copy link
Collaborator

Choose a reason for hiding this comment

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

List of creators to be updated after the current PR v2.1

@msdemlei
Copy link
Contributor Author

msdemlei commented May 18, 2020 via email

@msdemlei
Copy link
Contributor Author

msdemlei commented May 18, 2020 via email

@Zarquan
Copy link
Member

Zarquan commented May 20, 2020

Does anyone use that? If so, could they perhaps make a quick IVOA note on it?

I agree a note on how it is used would be good.

Copy link
Member

@Zarquan Zarquan left a comment

Choose a reason for hiding this comment

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

If you are OK with the changes I added to the descriptions then I'd say this is good to go.
@baptiste are you happy ?

@baptiste
Copy link

If you are OK with the changes I added to the descriptions then I'd say this is good to go.
@baptiste are you happy ?

I have no opinion but please @ your collaborator

@BaptisteCecconi
Copy link
Collaborator

Sorry to the other baptiste !
I'm happy with the changes, provided that we update the author list once v2.1 PR is approved.

@Zarquan
Copy link
Member

Zarquan commented May 22, 2020

I have no opinion but please @ your collaborator

Apologies for @'ing a random bystander.

Copy link
Member

@Zarquan Zarquan left a comment

Choose a reason for hiding this comment

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

Go for it 👍

@Zarquan Zarquan merged commit 987dfe7 into ivoa-std:master May 22, 2020
Zarquan pushed a commit that referenced this pull request May 7, 2023
BaptisteCecconi added a commit that referenced this pull request May 7, 2025
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.

4 participants