Skip to content

Conversation

jordancundiff
Copy link

This is to address issue number #666

@APCox
Copy link
Contributor

APCox commented Sep 10, 2025

Change the commit to be to the ArtifactOntology.ttl instead of the merged file and this looks good.

@APCox APCox added the Reviewed by CCO WG This item has been reviewed by participants of the CCO WG. label Sep 10, 2025
@cameronmore
Copy link
Contributor

I think @alanruttenberg has some ideas about changing this in conjunction with some other material, so I want to loop him in here.

@jordancundiff
Copy link
Author

these were the other items discussed to address (which we talked about in yesterday's WG) which I need to add to this PR:

  • Create Display Instrument as a sibling to Imaging Instrument.
  • Create Image Display Artifact Function as a sibling to Imaging Artifact Function.
  • Clarify the difference between "imaging" and "displaying an image".
  • Clarify the meaning of "produce images" in the definition of Imaging Instrument.

But I can hold off on continuing with this if there's more to be done.

@APCox
Copy link
Contributor

APCox commented Sep 16, 2025

@cameronmore

I think @alanruttenberg has some ideas about changing this in conjunction with some other material, so I want to loop him in here.

Alan was part of the discussion leading up to this PR. It was agreed that additional related fixes will be made in other PRs.

@APCox APCox moved this to In Progress in CCO Mission Sprint Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed by CCO WG This item has been reviewed by participants of the CCO WG.
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants