Skip to content

Align os.utime float-to-duration rounding with CPython semantics #6501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Copilot wants to merge 11 commits into from
+30 −4
Closed

Align os.utime float-to-duration rounding with CPython semantics #6501

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
fd73a9c
Initial plan
Copilot Dec 25, 2025
da24a78
Fix duration conversion rounding for os.utime
Copilot Dec 25, 2025
b911e6f
Address review feedback for duration rounding
Copilot Dec 25, 2025
7578fd9
Refine duration conversion constants and checks
Copilot Dec 25, 2025
31765b7
Clarify float nanosecond conversion cast
Copilot Dec 25, 2025
56acf0a
Use checked conversion for float nanoseconds
Copilot Dec 25, 2025
514b716
Add explicit float bounds before nanosecond cast
Copilot Dec 25, 2025
4c3e73f
Streamline duration bounds validation
Copilot Dec 25, 2025
73aa693
Convert float nanoseconds with checked filtering
Copilot Dec 25, 2025
efea81d
Use explicit bounds check before casting nanoseconds
Copilot Dec 25, 2025
a781c0d
Check nanosecond sign before duration bounds
Copilot Dec 25, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use checked conversion for float nanoseconds 
Co-authored-by: youknowone <69878+youknowone@users.noreply.github.com>
  • Loading branch information
@youknowone
Copilot and youknowone committed Dec 25, 2025
commit 56acf0ac7cf0897ddc0e2f0b2bea6ee84af702a7
7 changes: 4 additions & 3 deletions crates/vm/src/convert/try_from.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,12 +162,13 @@ fn duration_from_f64_floor(f: f64, vm: &VirtualMachine) -> PyResult<std::time::D
}

let total_nanos = (f * NANOS_PER_SEC).floor();
if total_nanos > MAX_TOTAL_NANOS as f64 {
let total_nanos_u128 = total_nanos
.to_u128()
.ok_or_else(|| vm.new_value_error("value out of range"))?;
if total_nanos_u128 > MAX_TOTAL_NANOS {
return Err(vm.new_value_error("value out of range"));
}

// safe: total_nanos is finite, non-negative, and below MAX_TOTAL_NANOS
let total_nanos_u128 = total_nanos as u128;
let secs = (total_nanos_u128 / NANOS_PER_SEC_U128) as u64;
let nanos = (total_nanos_u128 % NANOS_PER_SEC_U128) as u32;

Expand Down