Skip to content

Convert time scale for OME-TIFF#308

Merged
ziw-liu merged 4 commits into
mainfrom
ome-tiff-time-scale
Jun 17, 2025
Merged

Convert time scale for OME-TIFF#308
ziw-liu merged 4 commits into
mainfrom
ome-tiff-time-scale

Conversation

@ziw-liu
Copy link
Copy Markdown
Contributor

@ziw-liu ziw-liu commented Jun 3, 2025

NDTiff does not seem to carry the same information. The unit is always in seconds to prevent mismatch in automated analysis.

Related: #103, #304.

@ziw-liu ziw-liu requested review from edyoshikun and ieivanov June 3, 2025 23:30
@ziw-liu ziw-liu added this to the 0.2.1 milestone Jun 3, 2025
@ziw-liu ziw-liu requested a review from talonchandler June 3, 2025 23:31
@ziw-liu ziw-liu requested a review from aliddell June 17, 2025 20:41
Copy link
Copy Markdown
Contributor

@ieivanov ieivanov left a comment

Choose a reason for hiding this comment

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

In 359a868 I moved around the t_scale property to make the definition consistent with zyx_scale and avoid overwriting inherited properties where it's not needed. I think this PR looks good.

@ziw-liu ziw-liu merged commit 12e5023 into main Jun 17, 2025
7 checks passed
@ziw-liu ziw-liu deleted the ome-tiff-time-scale branch June 17, 2025 23:37
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.

2 participants