proposal-decimal icon indicating copy to clipboard operation
proposal-decimal copied to clipboard

Rounding mode "halfExpand" instead of "halfCeil"

Open andreasunger opened this issue 1 year ago • 1 comments

Assuming that the rounding mode "halfEven" corresponds to the IEEE rounding mode "roundTiesToEven", it looks like "halfCeil" is meant to correspond to "roundTiesToAway". However, because "ceiling" usually means rounding "toward +∞", should "halfCeil" rather be "halfExpand" (i.e., "expand" == "away from zero")?

andreasunger avatar May 04 '24 12:05 andreasunger

It looks like the proposal was updated to use the official rounding mode names. This removes the confusion.

One thing to note is that it now does not align with the Temporal rounding mode names anymore. I wish they would have used the official names as well.

andreasunger avatar May 11 '24 14:05 andreasunger

Right, we now use halfExpand rather than halfCeil, which is unsupported. That was a mistake on my part; sorry for the confusion.

jessealama avatar May 17 '24 14:05 jessealama

Since the rounding mode names were changed back to align with the existing names in the spec, I'm closing this issue.

andreasunger avatar May 17 '24 23:05 andreasunger