Skip to content

Remove StartTime-ms metadata tag from cameras#279

Merged
marktsuchida merged 1 commit intomainfrom
remove-start-time-ms
Oct 27, 2022
Merged

Remove StartTime-ms metadata tag from cameras#279
marktsuchida merged 1 commit intomainfrom
remove-start-time-ms

Conversation

@marktsuchida
Copy link
Copy Markdown
Member

In most cameras, this tag contained the value of GetCurrentMMTime() at the beginning of sequence acquisition. That value is milliseconds since 2000-01-01 UTC, plus an offset dependent on the current time zone.

The utility of this tag is therefore not clear: it is not documented anywhere, its interpretation is complicated, and there is no evidence that it has ever been used in MMStudio or its plugins.

If we ever find a need for recording the time of start of acquisition, it is better to add a tag in MMCore so that all cameras have it, not just the ones that happened to copy-paste program this tag.

In most cameras, this tag contained the value of `GetCurrentMMTime()` at
the beginning of sequence acquisition. That value is milliseconds since
2000-01-01 UTC, plus an offset dependent on the current time zone.

The utility of this tag is therefore not clear: it is not documented
anywhere, its interpretation is complicated, and there is no evidence
that it has ever been used in MMStudio or its plugins.

If we ever find a need for recording the time of start of acquisition,
it is better to add a tag in MMCore so that all cameras have it, not
just the ones that happened to copy-paste program this tag.
@marktsuchida marktsuchida mentioned this pull request Oct 26, 2022
5 tasks
@marktsuchida
Copy link
Copy Markdown
Member Author

See this previous comment for some more details.

@marktsuchida marktsuchida merged commit f27b395 into main Oct 27, 2022
@marktsuchida marktsuchida deleted the remove-start-time-ms branch October 27, 2022 20:30
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.

1 participant