Closed yutannihilation closed 4 years ago
速度比較しようと思ったけど今のmasterだとbench::mark()
が終わらなかった...。ということでたぶんめっちゃ速くなっているはずです。
library(zipangu)
set.seed(100)
d <- as.Date(sample(1:50000, 100, replace = TRUE), origin = "1970-01-01")
bench::mark(
is_jholiday(d),
min_iterations = 20
)
#> Warning: Some expressions had a GC in every iteration; so filtering is disabled.
#> # A tibble: 1 x 6
#> expression min median `itr/sec` mem_alloc `gc/sec`
#> <bch:expr> <bch:tm> <bch:tm> <dbl> <bch:byt> <dbl>
#> 1 is_jholiday(d) 248ms 256ms 3.90 NA 11.1
Created on 2020-04-05 by the reprex package (v0.3.0)
@uribo レビュー可能になったのでお手すきのとき見ていただければ...
@yutannihilation ありがとうございます。素晴らしいです! NEWSの追加とDESCRIPTIONにコントリビュータとしての追記をお願いできますか?
お、了解です。やります。
Thanks a lot :)
Thanks!
ちなみに year
が複数もあるようになったことで、 #16 の実装は若干ややこしくなる気がしています。今日は手がつけられなかったけど近々やります...
Related: #15
以下の関数に複数の
year
を渡せるようになります。find_date_by_wday()
jholiday_spec()
jholiday()
is_jholiday()
(これはすでに https://github.com/uribo/zipangu/commit/3c01ca58a4cf7676b448ed0983d8819aedf474f4 で対応済みだけど、速くなったはず)使用例
Created on 2020-04-05 by the reprex package (v0.3.0)