chaos/NTP-Timeturner
1
0
Fork 0
mirror of https://github.com/cjfranko/NTP-Timeturner.git synced 2025-11-08 18:32:02 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: prevent float precision loss in timecode calculation

Browse source 
Co-authored-by: aider (gemini/gemini-2.5-pro-preview-05-06) <aider@aider.chat>
This commit is contained in:
Chaos Rogers 2025-07-31 20:27:01 +01:00
parent a8aaf861ef
commit 30bcfe61d7
1 changed files with 11 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

24
src/system.rs
View file

@ -39,19 +39,17 @@ pub fn ntp_service_toggle(start: bool) {
pub fn calculate_target_time(frame: &LtcFrame, config: &Config) -> DateTime<Local> {
let today_local = Local::now().date_naive();
// Calculate total milliseconds including fractional frames
let total_ms = ((frame.hours as f64 * 3600.0 +
frame.minutes as f64 * 60.0 +
frame.seconds as f64 +
frame.frames as f64 / frame.frame_rate) * 1000.0).round() as u32;
let seconds = total_ms / 1000 % 60;
let minutes = (total_ms / 60000) % 60;
let hours = (total_ms / 3600000) % 24;
let ms_component = total_ms % 1000;
let timecode = NaiveTime::from_hms_milli_opt(hours, minutes, seconds, ms_component)
.expect("Invalid LTC timecode");
// Calculate millisecond component from frames with high precision, keeping H:M:S as integers.
let ms_from_frames = (frame.frames as f64 / frame.frame_rate * 1000.0).round() as u32;
let timecode = NaiveTime::from_hms_milli_opt(
frame.hours,
frame.minutes,
frame.seconds,
ms_from_frames,
)
.expect("Invalid LTC timecode");
let naive_dt = today_local.and_time(timecode);
let mut dt_local = Local