Skip to content

Add a content-type for the source code for NEST extension modules#272

Open
apdavison wants to merge 2 commits intomainfrom
add-ct-nest-model-cpp
Open

Add a content-type for the source code for NEST extension modules#272
apdavison wants to merge 2 commits intomainfrom
add-ct-nest-model-cpp

Conversation

@apdavison
Copy link
Copy Markdown
Member

(as produced by NESTML, for example)

@apdavison apdavison requested a review from lzehl March 12, 2025 14:45
Comment thread instances/latest/contentTypes/application_vnd.nest-simulator.model+cpp.jsonld Outdated
Comment thread instances/latest/contentTypes/application_vnd.nest-simulator.model+cpp.jsonld Outdated
@lzehl
Copy link
Copy Markdown
Member

lzehl commented Mar 14, 2025

Note : if the auto population is not merged yet by time of merging this PR we need to provide this CT also for the other versions

@apdavison apdavison added the ContentType ContentType instances - addition of new or update of existing ones label Mar 14, 2025
],
"name": "application/vnd.nest-simulator.model+cpp",
"relatedMediaType": "text/x-c",
"relatedMediaType": "https://mimetype.io/text/x-c",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

mh... this seems not be an official media type registered in iana.org

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would probably then not provide this relation at all. Could you double check?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I couldn't find an entry at iana.org - the initial "x-" generally indicates a non-standard (but widely used) media type.

Since the number of media types registered in iana.org is so small, I would like to propose relaxing our requirements for this property. mimetype.io seems like quite a comprehensive resource; I think we should also allow putting other openMINDS content types here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this might be point for v5.0. could you raise this in an issue please?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

"name": "application/vnd.nest-simulator.model+cpp",
"relatedMediaType": "https://mimetype.io/text/x-c",
"specification": "https://nest-extension-module.readthedocs.io/en/latest/extension_modules.html",
"synonym": null
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is there a more human readable name?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I can't think of anything that isn't just rephrasing the description, i.e. "Source code for an extension module for the NEST simulator"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ContentType ContentType instances - addition of new or update of existing ones

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants