関数メソッドのリファクタリング

gogotanaka / hilbert

:dancers: Implement mathematics.

http://hilbert-lang.org/

553 stars 36 forks source link

Closed huyu398 closed 10 years ago

huyu398 commented 10 years ago

値を求める場合に， e を Math::E に置換する際， ' e ' としているのは 2e-10 のような指数表記の e と区別するためです．

bundle exec rake spec を実行したところ， 309 個すべてのテストをクリアしました．

gogotanaka commented 10 years ago

@huyu398 すごいです！！！ありがとうございます！！！

そのTODO: Refactor を僕が考えていたのと同じ様に（それ以上に）Refactorして下さり本当に、感激しております.

huyu398 commented 10 years ago

@gogotanaka OSS project への pull request は初めてだったので不安でしたが，喜んでいただけたようで嬉しいです．とても面白い gem だと思いますので，今後も協力できればと思います．

gogotanaka commented 10 years ago

@huyu398 本当に嬉しいです. 自分はOSSへのcommit経験こそそこそこありますが、

自分のプロジェクトに大きく貢献して下さってくれる人が現れる経験はないので、、、

嬉しいです！

コーディンやコメントを拝見する限り、かなりrubyは書け、数学の見識も深い方だと察しますに、collaborator権限をお渡ししてもいいかなと思っております.

huyu398 commented 10 years ago

@gogotanaka 恐縮です……！！実は Ruby は元より OOP を本格的に取り組んだのはここ最近で，数学も工学科特有の現実的な問題が起こらなければ良しとする程度の厳密でないものしか習得できていませんが，そちらが宜しければ是非ともご協力したいと思います．

gogotanaka commented 10 years ago

@huyu398 いえいえ、僕は逆に純数学家な分、数値積分や近似用いた数学を苦手としているので助かります.

むしろ、このgemではその工学的な現実問題を解決する手法が必要だと考えています.

gogotanaka commented 10 years ago

@huyu398 ruby最近始めたってすごいです... センスありますよ！

gogotanaka commented 10 years ago

@huyu398 お疲れま様です. :sparkles:

ver 0.1.3としてリリースしました. gem update で手に入ります.