-
Notifications
You must be signed in to change notification settings - Fork 56
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
NXenvironment #1451
Comments
PR #1414 |
Proposal:
|
Check the nxdl.xsd Schema for the default attribute. If it is there for all
fields and groups, it is not necessary to add the NXobject. Lots of work is
done in the nxdl.xsd Schema.
…On Sun, Sep 29, 2024, 1:47 PM sanbrock ***@***.***> wrote:
Proposal:
- use of @default <https://github.com/default>
A) add to each group manually as proposed
B) no need to add, because it is a NeXus conventions that all groups
can have this attribute
C) add this to NXobject and let all group inherit it
- NXidentifier
A) use as a Base class which we add wherever needed
B) make the Fields to Attributes and use this base class as super
class. This allows attaching a single identifier to any group inheriting
from this.
C) same as B, but use nameType=partial, like identifier_serviceNAME,
identifierNAME, and identifier_is_persistantNAME to support multiple IDs to
add. E.g. identifier_experiment, identifier_collection, identifier_orcid,
identifier_pdb
- NXfabrication
features: vendor, ***@***.*** <https://github.com/Version>),
serial_number, construction_year, capability, IDENTIFIER
- NXsensor
added 'current' to the enum list of measurement types
features FABRICATION
- NXactuator
features: (short)name, type, FABRICATION, physical_quantity, pid,
OUTPUT, TRANSFORMATIONS
- NXenvironment:
added ACTUATOR
added 'value' to support the case where no actual SENSOR, ACTUATOR is
attached
—
Reply to this email directly, view it on GitHub
<#1451 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARMUMB7HPKLALZ7ADUX5RLZZBDMRAVCNFSM6AAAAABO437CEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGQ2TSNJQHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Use of default: B) |
Proposal: Use of default: B) The above is approved by vote in NIAC 2024 Not for this proposal: |
The text was updated successfully, but these errors were encountered: