Update UCD section relative to last discussion with Semantics WG by loumir · Pull Request #61 · ivoa/HighEnergyObsCoreExt

loumir · 2026-05-27T17:54:26Z

I would like the results of the semantics WG discussions that happened to appear in this note. This was part of the work all along the year.
Therefore in the listed requirements for UCD we should use the terms that have been recently discussed and approved by the semantics group .
I have updated various parts of this section and included %comments to explain the changes .

…ics WG

Clarified the usage of Particle Data Group Identifier for particle detection and the electron case.

loumir

Please review the UCD changes, a few inconsistencies may still be hidden.
thanks

showing the result of recent discussions with Semantics WG

loumir

thanks for your careful reading.
many places changed due to the UCD related elements .

The different roles between the column names , using the dedicated language of the specific domain , here high energy , and the UCD terms using a more general language to foster crosswalks between the various usage in other spectral domains , should be clear in the note.

iannevans

Added a few comments and recommended changes where I thought they made sense.

iannevans · 2026-05-27T20:33:26Z

-P & {\em stat.error.positive\/}  & Positive statistical error  \cr
-P & {\em stat.lowerlimit\/}  & Lower limit \cr
-P & {\em stat.upperlimit\/}  & Upper limit \cr
+% Mireille we cannot have these terms in the UCD tree ; that would imply importing all possible encoding of any kind 


I disagree about phys.particle.pdgid. phys.particle.pdgid is a UCD that identifies a Particle Group Identifier, for example a column in a FITS file that records the pdgid for each event. It's not the same as UCDs that encode all possible pdgids as part of the UCD name.

Ok for me Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

+1 Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Updated UCD descriptions for event grade, type, and physical quantities in HighEnergyObsCoreExt.tex.

loumir

I hope I filled the changes . I am not used to the GitHub on line editor , so what is taken into account for the PDF compilation is not clear to me.

loumir

test

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

bkhelifi

Thanks a lot to all! OK for me

bkhelifi · 2026-06-02T13:19:51Z

@iannevans: a very last short question about proposal_id (section 3.11). When we have a list of IDs, should we recommand to use # as separator? If you agree, you can make directly the change and do the merge!

iannevans · 2026-06-02T15:24:42Z

@iannevans: a very last short question about proposal_id (section 3.11). When we have a list of IDs, should we recommand to use # as separator? If you agree, you can make directly the change and do the merge!

We want to bring proposal_id into line with facility_name and instrument_name, which are the other two "provenance" attributes in ObsCore. They already support multiple values. For facility_name, the REC says "For combined observations stemming from multiple facilities the name may contain a list of comma separated strings, or the word "Many"; if the list is too long, as defined in the VODataservice specification." and instrument_name states "Multiple values are also allowed for complex observations as defined for facility name."

For proposal_id, could recommending adding the same sentence (i.e.,) "Multiple values are also allowed for complex observations as defined for facility name." but I'm not convinced it's necessary.

iannevans · 2026-06-02T15:24:45Z

@iannevans: a very last short question about proposal_id (section 3.11). When we have a list of IDs, should we recommand to use # as separator? If you agree, you can make directly the change and do the merge!

We want to bring proposal_id into line with facility_name and instrument_name, which are the other two "provenance" attributes in ObsCore. They already support multiple values. For facility_name, the REC says "For combined observations stemming from multiple facilities the name may contain a list of comma separated strings, or the word "Many"; if the list is too long, as defined in the VODataservice specification." and instrument_name states "Multiple values are also allowed for complex observations as defined for facility name."

For proposal_id, could recommending adding the same sentence (i.e.,) "Multiple values are also allowed for complex observations as defined for facility name." but I'm not convinced it's necessary.

iannevans · 2026-06-02T19:19:19Z

 \paragraph{Event Type}

-For \gls{VHE} (and GeV) gamma-ray data, there is the notion of event type (see \S~\ref{sec:evttype}) that can be mandatory for some data releases.  We propose to add a new UCD {\em instr.event.type\/} that identifies these data values.
+For \gls{VHE} (and GeV) gamma-ray data, there is the notion of event type (see \S~\ref{sec:evttype}) that can be mandatory for some data releases.  We propose to use a combined UCD {\em meta.code.class;instr.detection\/} to tag the proposed {\bf event\_type} parameter.


Would "to tag event type" make more sense? Similar comment to "event_grade": assuming all instruments/facilities use "event_type" as a column name is too optimistic.

iannevans

I made a few suggestions but this is definitely converging. There's a question for Bruno hidden in my comment on the description of phys.pulse ;-)

loumir · 2026-06-11T09:48:27Z

 That could be solved by marking {\em phys.electron\/} (and {\em phys.electron.degen\/}) as obsolete or not recommended, and adding the term  {\em phys.particle.electron\/} to the UCD list.
 However, as many data sets in the IVOA projects are already tagged with {\em phys.electron\/} this may require maintenance costs that should be evaluated . 
 It will be up to the IVOA to evaluate the benefice/cost ratio of such a change. 
 %The UCD term {\em phys.electron\/} is available anyway and can be used to tag an event related to electron particles.


following the discussion with semantics on June 10th , we can keep this line to indicate the electron case is covered anyway.

as discussed in last semantics -Heig meeting in Strasbourg , June 10th

loumir

last commits include the changes decided in Strasbourg meeting on June 10th.

get vep files in update_semantics_part

loumir and others added 3 commits May 27, 2026 19:46

Update UCD discussion section relative to last discussion with semant…

5815b90

…ics WG

Update UCD terms and particle identifier usage

a22fef9

Clarified the usage of Particle Data Group Identifier for particle detection and the electron case.

Update UCDs from Semantics WG discussion part2

2120e6e

loumir commented May 27, 2026

View reviewed changes

loumir added 2 commits May 27, 2026 20:38

Update UCD terms for event-list and pulse height

36aabbe

showing the result of recent discussions with Semantics WG

Update UCD for event as instr.detection

bd8c626

loumir commented May 27, 2026

View reviewed changes

iannevans requested changes May 27, 2026

View reviewed changes

loumir and others added 5 commits May 28, 2026 15:33

Update HighEnergyObsCoreExt.tex

414356e

Ok for me Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

typos

befb151

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

formalizing line 540

0de2202

+1 Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

l 172

f8559c9

+1 Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Refine UCD terms for event properties in HEA data

089fa73

Updated UCD descriptions for event grade, type, and physical quantities in HighEnergyObsCoreExt.tex.

loumir commented May 28, 2026

View reviewed changes

Fix formatting of event_type parameter in LaTeX file

bc65706

loumir commented May 28, 2026

View reviewed changes

loumir and others added 2 commits June 2, 2026 11:33

update with suggestion about phys.particle.electron

d7d697d

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

9c64623

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

loumir requested a review from iannevans June 2, 2026 09:38

bkhelifi and others added 10 commits June 2, 2026 12:09

Update HighEnergyObsCoreExt.tex

bda8f9d

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

99190a4

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

227c20e

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

a472252

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

b7460ee

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

4ab6409

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

8e3d36c

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

ae4cb3a

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

ffa83f5

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

Update HighEnergyObsCoreExt.tex

a497f98

Co-authored-by: Ian Nigel Evans <ievans@cfa.harvard.edu>

bkhelifi approved these changes Jun 2, 2026

View reviewed changes

iannevans reviewed Jun 2, 2026

View reviewed changes

Comment thread HighEnergyObsCoreExt.tex Outdated

iannevans reviewed Jun 2, 2026

View reviewed changes

Comment thread HighEnergyObsCoreExt.tex Outdated

iannevans reviewed Jun 2, 2026

View reviewed changes

Comment thread HighEnergyObsCoreExt.tex Outdated

iannevans reviewed Jun 2, 2026

View reviewed changes

loumir commented Jun 11, 2026

View reviewed changes

loumir added 2 commits June 15, 2026 17:39

Rename event-list to he-event-list in documentation

ff6cc66

Update instr.detection as instr.detection;phys.particle

5274373

as discussed in last semantics -Heig meeting in Strasbourg , June 10th

loumir commented Jun 15, 2026

View reviewed changes

loumir and others added 4 commits June 16, 2026 12:11

Merge pull request #2 from ivoa/main

c9f8e87

get vep files in update_semantics_part

update hea-event-list and hea-event-bundle

322fb32

cleaning

e3743c4

some Ian's comments

ca73877

bkhelifi merged commit 9d79952 into ivoa:main Jun 16, 2026

Conversation

loumir commented May 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

loumir left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

loumir left a comment

Choose a reason for hiding this comment

Uh oh!

iannevans left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

iannevans May 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

loumir left a comment

Choose a reason for hiding this comment

Uh oh!

loumir left a comment

Choose a reason for hiding this comment

Uh oh!

bkhelifi left a comment

Choose a reason for hiding this comment

Uh oh!

bkhelifi commented Jun 2, 2026

Uh oh!

iannevans commented Jun 2, 2026

Uh oh!

iannevans commented Jun 2, 2026

Uh oh!

Uh oh!

Uh oh!

iannevans Jun 2, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

iannevans left a comment

Choose a reason for hiding this comment

Uh oh!

loumir Jun 11, 2026

Choose a reason for hiding this comment

Uh oh!

loumir left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

loumir commented May 27, 2026 •

edited

Loading

loumir left a comment •

edited

Loading