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
Refine duration conversion constants and checks 
Co-authored-by: youknowone <69878+youknowone@users.noreply.github.com>
  • Loading branch information
@youknowone
Copilot and youknowone committed Dec 25, 2025
commit 7578fd9f6bd49aff245be96986e9fa23ae74a3bf
5 changes: 4 additions & 1 deletion crates/vm/src/convert/try_from.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,8 +148,8 @@ impl TryFromObject for std::time::Duration {
}

fn duration_from_f64_floor(f: f64, vm: &VirtualMachine) -> PyResult<std::time::Duration> {
const NANOS_PER_SEC: f64 = 1_000_000_000.0;
const NANOS_PER_SEC_U128: u128 = 1_000_000_000;
const NANOS_PER_SEC: f64 = NANOS_PER_SEC_U128 as f64;
// Maximum duration representable by (secs, nanos) with u64 seconds and <1e9 nanoseconds.
const MAX_TOTAL_NANOS: u128 =
((u64::MAX as u128) * NANOS_PER_SEC_U128) + (NANOS_PER_SEC_U128 - 1);
Expand All @@ -162,6 +162,9 @@ fn duration_from_f64_floor(f: f64, vm: &VirtualMachine) -> PyResult<std::time::D
}

let total_nanos = (f * NANOS_PER_SEC).floor();
if total_nanos < 0.0 {
return Err(vm.new_value_error("negative duration"));
}
if total_nanos > MAX_TOTAL_NANOS as f64 {
return Err(vm.new_value_error("value out of range"));
}
Expand Down