ichimomo / frasyr

Fisheries Research Agency (FRA) provides the method for calculating sustainable yield (SY) with R
https://ichimomo.github.io/frasyr/
GNU General Public License v3.0
16 stars 21 forks source link

specify.Mesnil.gamma #701

Open ichimomo opened 2 years ago

ichimomo commented 2 years ago

上の関数,check.SRfitとかなりコードが重複しているみたいなので,できるだけコードのコピーは避けて,同じコードを使う(たとえばspecify.Mesnil.gammaの中でcheck.SRfitを呼び出す)ことはできないでしょうか? 複数の同じ(ような)コードは,どうしてもバグの原因になるので

fshin3 commented 2 years ago

check.SRfitもいじる感じでいいですか?全部チェックするわけじゃないのと、該当部のメッセージを全部出す必要もないので。

ichimomo commented 2 years ago

全然良いですよ 共同開発っていうのはそういうもんです

2021年11月25日(木) 10:10 Shin Fukui @.***>:

check.SRfitもいじる感じでいいですか?全部チェックするわけじゃないのと、該当部のメッセージを全部出す必要もないので。

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ichimomo/frasyr/issues/701#issuecomment-978690357, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKQB4KYPMZNNEBJZ42G7KDTUNWEHZANCNFSM5IXKXK2A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- 市野川 (安部) 桃子 Ichinokawa (Abe), Momoko @.*** https://ichimomo.github.io/main/ 水産研究・教育機構 水産資源研究所 水産資源研究センター 漁業情報解析部 資源解析グループ長 〒236-8648 神奈川県横浜市金沢区福浦2-12-4 Tel. 045-788-7920 (直) 045-788-7615 (代)

ichimomo commented 2 years ago

改変しても,テストがちゃんと機能していれば大丈夫なはずなので.なので,改変する関数について,ちゃんとテストコードが書かれているかは事前に確認してください

2021年11月25日(木) 10:14 Momoko Ichinokawa @.***>:

全然良いですよ 共同開発っていうのはそういうもんです

2021年11月25日(木) 10:10 Shin Fukui @.***>:

check.SRfitもいじる感じでいいですか?全部チェックするわけじゃないのと、該当部のメッセージを全部出す必要もないので。

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ichimomo/frasyr/issues/701#issuecomment-978690357, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKQB4KYPMZNNEBJZ42G7KDTUNWEHZANCNFSM5IXKXK2A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- 市野川 (安部) 桃子 Ichinokawa (Abe), Momoko @.*** https://ichimomo.github.io/main/ 水産研究・教育機構 水産資源研究所 水産資源研究センター 漁業情報解析部 資源解析グループ長 〒236-8648 神奈川県横浜市金沢区福浦2-12-4 Tel. 045-788-7920 (直) 045-788-7615 (代)

-- 市野川 (安部) 桃子 Ichinokawa (Abe), Momoko @.*** https://ichimomo.github.io/main/ 水産研究・教育機構 水産資源研究所 水産資源研究センター 漁業情報解析部 資源解析グループ長 〒236-8648 神奈川県横浜市金沢区福浦2-12-4 Tel. 045-788-7920 (直) 045-788-7615 (代)