OpenTimelineIO icon indicating copy to clipboard operation
OpenTimelineIO copied to clipboard

Published 20 hours ago • verified publisher icon AcademySoftwareFoundation

Fix RationalTime Representation

Open visajshah opened this issue 4 years ago • 3 comments

Fixes #682 RationalTime Representation

visajshah avatar Sep 22 '21 05:09 visajshah

Thanks for taking a look at this! Could you add some tests also to show that the intended use case (e.g. round-trip from RationalTime to string to back again) are working properly? Also, there is some background information and suggested strategy for this here: https://stackoverflow.com/questions/16839658/printf-width-specifier-to-maintain-precision-of-floating-point-value/19897395

jminor avatar Sep 22 '21 15:09 jminor

Yeah, sure, I will look into it. I will also have to look at the build tests and clear that.

visajshah avatar Sep 23 '21 13:09 visajshah

@visajshah Are you still interested in pursuing this one? The intention seemed worthwhile.

meshula avatar Sep 18 '22 19:09 meshula

Closing due to inactivity.

meshula avatar Apr 18 '24 20:04 meshula