Skip to content

added mikro seconds (us) and nano seconds (ns) as durations in units.yaml#855

Open
DavidMatzek wants to merge 2 commits into
COVESA:masterfrom
DavidMatzek:added/units_time
Open

added mikro seconds (us) and nano seconds (ns) as durations in units.yaml#855
DavidMatzek wants to merge 2 commits into
COVESA:masterfrom
DavidMatzek:added/units_time

Conversation

@DavidMatzek
Copy link
Copy Markdown

About

At BMW we develop a VSS - model prototype for odometry. We want to display some timestamps in nanoseconds.

I want COVESA to include the needed units in case this prototype is successful and we start real implementation.

Clarification of abbreviations.

To me it was not clear how to express the Greek alphabet, here is what i found:
Wikipedia - Mikrosecond
{AC428A24-62AE-48F5-B165-B5428A94E526}

Signed-off-by: David Matzek <david.dm.matzek@bmw.de>
Comment thread spec/units.yaml Outdated
@erikbosch
Copy link
Copy Markdown
Collaborator

MoM:

Copy link
Copy Markdown
Collaborator

@erikbosch erikbosch left a comment

Choose a reason for hiding this comment

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

Change to micro

@erikbosch
Copy link
Copy Markdown
Collaborator

@DavidMatzek - will you do the requested update?

@DavidMatzek
Copy link
Copy Markdown
Author

Sorry for delay, changed mikro to micro :).

@DavidMatzek DavidMatzek requested a review from erikbosch February 4, 2026 07:46
@erikbosch
Copy link
Copy Markdown
Collaborator

Thanks for the update, the build error seems to be an unrelated regression due to a change in s2dm, will check it out.

Comment thread spec/units.yaml
allowed-datatypes: ['numeric']
us:
definition: Duration measured in microseconds
unit: mikrosecond
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sorry for the delay. Could you change to "microsecond" here as well and we shall be ready to go

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ping!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Comment thread spec/units.yaml
Comment on lines +175 to +184
ns:
definition: Duration measured in nanoseconds
unit: nanosecond
quantity: duration
allowed-datatypes: ['numeric']
us:
definition: Duration measured in microseconds
unit: mikrosecond
quantity: duration
allowed-datatypes: ['numeric']
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
ns:
definition: Duration measured in nanoseconds
unit: nanosecond
quantity: duration
allowed-datatypes: ['numeric']
us:
definition: Duration measured in microseconds
unit: mikrosecond
quantity: duration
allowed-datatypes: ['numeric']
ns:
definition: Duration measured in nanoseconds
unit: nanosecond
quantity: duration
allowed-datatypes: ['numeric']
qudt:
unit: http://qudt.org/vocab/unit/NanoSEC
quantity-kind: https://qudt.org/vocab/quantitykind/Time
us:
definition: Duration measured in microseconds
unit: microsecond
quantity: duration
allowed-datatypes: ['numeric']
qudt:
unit: https://qudt.org/vocab/unit/MicroSEC
quantity-kind: https://qudt.org/vocab/quantitykind/Time

@erikbosch
Copy link
Copy Markdown
Collaborator

MoM:

  • D: Updated, can merge this one after qudt
  • E: I can take care of conflicts

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