[ruby-core:91739] [Ruby trunk Feature#15653] Proposal: Add Time#floor

ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed

* [ruby-core:91739] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
@ 2019-03-11  3:38 ` manga.osyo
  2019-03-11  4:11 ` [ruby-core:91741] " akr
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: manga.osyo @ 2019-03-11  3:38 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been reported by osyo (manga osyo).

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91741] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
  2019-03-11  3:38 ` [ruby-core:91739] [Ruby trunk Feature#15653] Proposal: Add Time#floor manga.osyo
@ 2019-03-11  4:11 ` akr
  2019-03-11  6:26 ` [ruby-core:91752] " muraken
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: akr @ 2019-03-11  4:11 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by akr (Akira Tanaka).


良い機能だと思います。

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77017

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91752] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
  2019-03-11  3:38 ` [ruby-core:91739] [Ruby trunk Feature#15653] Proposal: Add Time#floor manga.osyo
  2019-03-11  4:11 ` [ruby-core:91741] " akr
@ 2019-03-11  6:26 ` muraken
  2019-03-11  6:29 ` [ruby-core:91753] " knu
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: muraken @ 2019-03-11  6:26 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by mrkn (Kenta Murata).


ActiveRecord を使うアプリケーションで、時刻をDBに入れて取り出すと秒の単位で切り捨てられます。
昔 Rails アプリケーションを作っていた頃は、この挙動のためにテストコードでの時刻の比較が面倒だったことを思い出しました。

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77028

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91753] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
                   ` (2 preceding siblings ...)
  2019-03-11  6:26 ` [ruby-core:91752] " muraken
@ 2019-03-11  6:29 ` knu
  2019-03-11  6:33 ` [ruby-core:91755] " matz
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: knu @ 2019-03-11  6:29 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by knu (Akinori MUSHA).


テストコードで `Time.at(time.to_i)` とかやることは多いですね。

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77029

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91755] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
                   ` (3 preceding siblings ...)
  2019-03-11  6:29 ` [ruby-core:91753] " knu
@ 2019-03-11  6:33 ` matz
  2019-03-11  6:39 ` [ruby-core:91757] " manga.osyo
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 8+ messages in thread
From: matz @ 2019-03-11  6:33 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by matz (Yukihiro Matsumoto).


Accepted.

Matz.


----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77031

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91757] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
                   ` (4 preceding siblings ...)
  2019-03-11  6:33 ` [ruby-core:91755] " matz
@ 2019-03-11  6:39 ` manga.osyo
  2019-03-11 14:13 ` [ruby-core:91783] " zverok.offline
  2019-03-11 14:27 ` [ruby-core:91784] " akr
  7 siblings, 0 replies; 8+ messages in thread
From: manga.osyo @ 2019-03-11  6:39 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by osyo (manga osyo).


レビューありがとうございます！

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77033

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91783] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
                   ` (5 preceding siblings ...)
  2019-03-11  6:39 ` [ruby-core:91757] " manga.osyo
@ 2019-03-11 14:13 ` zverok.offline
  2019-03-11 14:27 ` [ruby-core:91784] " akr
  7 siblings, 0 replies; 8+ messages in thread
From: zverok.offline @ 2019-03-11 14:13 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by zverok (Victor Shepelev).


Shouldn't there be also `ceil` for completeness, if we already have `round` and `floor`?

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77058

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:91784] [Ruby trunk Feature#15653] Proposal: Add Time#floor
       [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
                   ` (6 preceding siblings ...)
  2019-03-11 14:13 ` [ruby-core:91783] " zverok.offline
@ 2019-03-11 14:27 ` akr
  7 siblings, 0 replies; 8+ messages in thread
From: akr @ 2019-03-11 14:27 UTC (permalink / raw)
  To: ruby-core

Issue #15653 has been updated by akr (Akira Tanaka).


zverok (Victor Shepelev) wrote:
> Shouldn't there be also `ceil` for completeness, if we already have `round` and `floor`?

No one created a issue for that, yet.

I'm positive, though.

----------------------------------------
Feature #15653: Proposal: Add Time#floor
https://bugs.ruby-lang.org/issues/15653#change-77059

* Author: osyo (manga osyo)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
## 概要

`Time` の小数（ナノ秒）を指定した桁で切り捨てるメソッドの提案になります。


## 現状

* 小数を丸める `Time#round` はあるが小数を切り捨てるメソッドがない
* 精度が異なる時間を比較したり保存したい場合に小数を切り捨てたいケースがある
  * 例えば、小数の精度が3桁で扱われている DB からデータを取得したい場合など
  * Ruby 以外で時間に依存する場合に切り捨てや丸めに柔軟に対応したい


## 提案

* 指定した桁の小数を切り捨てる `Time#floor` メソッドの追加


## `Time#floor` の挙動

`Time#round` と同じような使い方になります。

```ruby
require 'time'

t = Time.utc(2010,3,30, 5,43,"25.123456789".to_r)
t.iso8601(10)           #=> "2010-03-30T05:43:25.1234567890Z"

# 引数に精度（桁数）を渡す
t.floor(0).iso8601(10)  #=> "2010-03-30T05:43:25.0000000000Z"
t.floor(1).iso8601(10)  #=> "2010-03-30T05:43:25.1000000000Z"
t.floor(2).iso8601(10)  #=> "2010-03-30T05:43:25.1200000000Z"
t.floor(3).iso8601(10)  #=> "2010-03-30T05:43:25.1230000000Z"
t.floor(4).iso8601(10)  #=> "2010-03-30T05:43:25.1234000000Z"
t.floor(5).iso8601(10)  #=> "2010-03-30T05:43:25.1234500000Z"
t.floor(6).iso8601(10)  #=> "2010-03-30T05:43:25.1234560000Z"
t.floor(7).iso8601(10)  #=> "2010-03-30T05:43:25.1234567000Z"
t.floor(8).iso8601(10)  #=> "2010-03-30T05:43:25.1234567800Z"
t.floor(9).iso8601(10)  #=> "2010-03-30T05:43:25.1234567890Z"
t.floor(10).iso8601(10) #=> "2010-03-30T05:43:25.1234567890Z"

# デフォルト引数は 0
t.floor.iso8601(10)     #=> "2010-03-30T05:43:25.0000000000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.4).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 0.49).floor.iso8601(3)   #=> "1999-12-31T23:59:59.000Z"
(t + 0.5).floor.iso8601(3)    #=> "1999-12-31T23:59:59.000Z"
(t + 1.4).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"
(t + 1.49).floor.iso8601(3)   #=> "2000-01-01T00:00:00.000Z"
(t + 1.5).floor.iso8601(3)    #=> "2000-01-01T00:00:00.000Z"

t = Time.utc(1999,12,31, 23,59,59)
(t + 0.123456789).floor(4).iso8601(6)  #=> "1999-12-31T23:59:59.123400Z"
```


## 問題点

次のケースで意図しない値が返って来ます。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.6 すると 1999-12-31T23:59:59.6000000000Z になる
# なので floor(1) を行うと 1999-12-31T23:59:59.6000000000Z になってほしいが .5 になって返ってくる
pp (t + 0.6).floor(1).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"

# これは + 0.6 した際に桁落ちしているのが影響している
pp (t + 0.6).iso8601(10)
# => "1999-12-31T23:59:59.5999999999Z"
```

また、 `Time#round` でも似たような挙動となっています。

```ruby
t = Time.utc(1999,12,31, 23,59,59)

# + 0.5 では意図する結果になる
pp (t + 0.5).round.iso8601(10)
# => "2000-01-01T00:00:00.0000000000Z"

# + 0.6 - 0.1 で計算すると意図しない結果になる
pp (t + 0.6 - 0.1).round.iso8601(10)
# => "1999-12-31T23:59:59.0000000000Z"

# これもナノ秒が以下のような値になっている為
pp (t + 0.5).iso8601(10)
# => "1999-12-31T23:59:59.5000000000Z"
pp (t + 0.6 - 0.1).iso8601(10)
# => "1999-12-31T23:59:59.4999999999Z"
```

現状ではこれが仕様かバグか判断出来なかったのでそのまま（上記の挙動）になっています。


pull request : https://github.com/ruby/ruby/pull/2092





-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-03-11 14:28 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <redmine.issue-15653.20190311033748@ruby-lang.org>
2019-03-11  3:38 ` [ruby-core:91739] [Ruby trunk Feature#15653] Proposal: Add Time#floor manga.osyo
2019-03-11  4:11 ` [ruby-core:91741] " akr
2019-03-11  6:26 ` [ruby-core:91752] " muraken
2019-03-11  6:29 ` [ruby-core:91753] " knu
2019-03-11  6:33 ` [ruby-core:91755] " matz
2019-03-11  6:39 ` [ruby-core:91757] " manga.osyo
2019-03-11 14:13 ` [ruby-core:91783] " zverok.offline
2019-03-11 14:27 ` [ruby-core:91784] " akr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).