From ce194c42d20a7a01f22b07ede3cf1aa12bd25688 Mon Sep 17 00:00:00 2001 From: GHYC-MacMini <103517188+UltraBlur@users.noreply.github.com> Date: Fri, 7 Nov 2025 10:16:18 +0800 Subject: [PATCH] Fix FCPX format output for fractional timestamps Fixed a bug in _convert_to_output_fcpx method where the denominator of fractional timestamps was missing the forward slash separator. Previously, fractional timestamps like Fraction(5, 2) would output as "52s" instead of the correct FCPX format "5/2s". This fix ensures the denominator includes the "/" prefix when the timestamp is not an integer, producing valid FCPX time format. --- dftt_timecode/core/dftt_timecode.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dftt_timecode/core/dftt_timecode.py b/dftt_timecode/core/dftt_timecode.py index 5ad1e96..642b2a0 100644 --- a/dftt_timecode/core/dftt_timecode.py +++ b/dftt_timecode/core/dftt_timecode.py @@ -684,7 +684,7 @@ def _convert_to_output_fcpx(self, output_part=0) -> str: else: logger.warning( '_convert_to_output_fcpx: This timecode type has only one part.') - output_fcpx_denominator='' if float(self.__precise_time).is_integer() else self.__precise_time.denominator + output_fcpx_denominator='' if float(self.__precise_time).is_integer() else f'/{self.__precise_time.denominator}' return f'{self.__precise_time.numerator}{output_fcpx_denominator}s' def _convert_to_output_frame(self, output_part=0) -> str: