Closed indenkun closed 4 years ago
素晴らしい実装をありがとうございます。 NEWSファイルに変更内容を簡潔に記載いただけますと幸いです。
また、差し支えなければパッケージのcontributorに加えさせていただきたく、DESCRIPTIONへの追加コミットをお願いできますでしょうか。 例) https://github.com/uribo/zipangu/commit/79feb301b9efd57c7dd11a1ffaf41ef862d4b0f9
ありがとうございます。
コード内に若干の不具合があったので修正とコード内でstats::na.omit
を使っているのでDescriptionのimportに{stats}
を追加しました。
テストも追加したほうが良いでしょうか。
対応いただきまして、ありがとうございます。
そうですね。@examples
に書いているコードを含めた簡単なテストを用意いただけますと助かります。
ありがとうございます。
自分でも、R-CMD-checkdeでエラーがでているのに気づいて直後に修正したコードをフォークしているリポジトリにはMergeしたのですが、うまくこのPull Requestに載せられておらず(自分のgithubの操作の理解が不十分なせいです)、すみません。確認中です。
https://github.com/uribo/zipangu/pull/23/commits/6ab6f3a6dd45e20fc846d84839c9d13f05f131a7 こちらのコミットで対応されていましたね!ありがとうございます。
Pull Requestできない理由がわからなかったので、ごくごく軽微なその他のファイルの修正(空行の削除)をマージしたら一緒に前のファイルもPull Request通りました。 お手数かけました。
Mergeしていただきありがとうございます。
漢数字(複数桁)の変換を含めて、漢数字からアラビア数字への変換について解決できそうな案(関数)について提案します。
kansuji2arabic_num
で京までの十や百などの位を表す漢数字を含む漢数字を既存のkansuji2arabic_all
では置換する形になるもの(例:二千二十が21000210になる)を、漢数字で表している数となるように計算するようになります(例:二千二十が2020になる)。kansuji2arabic_str
で文中の漢数字を上記のように計算した上でアラビア数字に変換する関数です。ContributingやPull request templateがなくどう提案するかわからなかったので、不適切な提案でしたら破棄してください。
8