uribo / zipangu

Japanese utility functions and data
https://uribo.github.io/zipangu
Other
56 stars 7 forks source link

add new function `label_kansuji()` and `label_kansuji_suffix()`. #27

Closed indenkun closed 3 years ago

indenkun commented 3 years ago

アラビア数字を漢数字に変換する{arabic2kansuji}がCRANに登録されたので、以前ISSUE #9 で提案した関数をそれぞれ label_kansuji()label_kansuji_suffix()として提案いたします。

label_kansuji()で漢字と数字混じりまたは漢数字のみのラベル、label_kansuji_suffix()で数字+漢数字のラベルに変換できるようにしています。

Importに{arabic2kansuji}を追加し、suggestsに{scales}を追加しています。{arabic2kansuji}を依存関係に追加した関係で他のコードを少し修正しています。

こんな感じで挙動しまします。

scales::demo_continuous(c(1, 1e9), label = label_kansuji())

Rplot03

scales::demo_continuous(c(1, 1e9), label = label_kansuji_suffix())

Rplot04

examplesはCRANポリシー的によくわからず不安があったのでdontrunにしています。

9

uribo commented 3 years ago

@indenkun ありがとうございます。対応が遅れてしまい、申し訳ありませんでした。 是非取り入れさせてください。 レビューコメントへの対応とNEWS.mdの New Features に追加した機能の概要を書いていただけますでしょうか。

indenkun commented 3 years ago

@uribo ご確認ありがとうございます。 レビューコメントありがとうございました。レビューコメントの通り、usethis::use_latest_dependencies()でバージョンについてそれぞれ記載し、news.mdのNew Featuresに漢数字ラベルについて追記しました。

お時間あるときにご確認いただければ幸いです。

uribo commented 3 years ago

ありがとうございます!そろそろCRANのバージョンをあげようと思います。