djangoアプリ再構築

[duri0214]

41 soil_analysis/chemical_report/models.py のテーブルの項目名などのチェックお願いします

レビューのポイント

• アルファベットや単語が合っているか？調べられる範囲で調べただけなので単語が違うとかがありえます（修正の必要があるときはコメントを書いてください。斜線（スラッシュ）とかは入れられません、英数字の範囲のみです。it業界的にはできるだけ英単語で書きますが、ローマ字のほうがわかりやすいこともあるのでそれはそれでokです）

• 整数型と小数点型が合っているか？（％で表現しそうなところは float にしておきました。1度でも小数点を使う可能性がある場合は float です。コメントを入れるときは「＋」マークから）

セットアップメモ

grant CREATE, DROP, SELECT, UPDATE, INSERT, DELETE, ALTER, REFERENCES, INDEX on soil_db.* to python@localhost;

• migrations フォルダを削除してから

  python manage.py makemigrations chemical_report
  python manage.py migrate
  python manage.py loaddata test_data1.json

[duri0214]

@yoshiyukiminami

つぎの土曜日やることリスト

• この３つ使ってなさそう。整理削除しても大丈夫ですか？
• models.py のレビューとすり合わせ
• 環境の再現を説明

[yoshiyukiminami]

岡田さん

上記の件ですが、 ・data_processing・・この2つのpyは使います。  data_preprocessing.py  tekusei_stats_cal.py ・graph・・気象庁HPから任意地点の気象データをスクレイピングするpyは使います。  test_scraping.py  test_scraping_csv.py  thisyaer_data_scraping.py  today_data_scraping.py basic_django_graph/mysiteは勉強用に作成してもらったやつで実際に使用する仕様のpyは作成していないので、整理は可能かと思います。

メールの返信しましたが、うまく返信できていない可能性がありそうなので、こちらでコメントしてみます

[yoshiyukiminami]

岡田さん

同じくGithubの方でうめくコメントできないので、コメントをコピペして送ります。 ＜以下、コメントコピペ＞

• [x] Class fieldsのcultivation_methodはclass_clopsの項目の方が良いかと思います。理由：露地・施設栽培と合わせてトンネル栽培など大きく分類すると「被覆する」かどうかは栽培品目と必ずリンクするので

• [x] ec 電気伝導度→電気伝導率 e.g. 1(mS/cm)

• [x] nh4n アンモニア性窒素→アンモニア態窒素 e.g. 1(mg/100g)

• [x] no3n 硝酸態窒素 e.g. 1(mg/100g)

• [x] inorganic_nitrogen→nh4+no3 無機態窒素 ??→無機態窒素（NH4＋NO3）

• [x] nh4→nh4/(nh4+no3) 無機態窒素 ??→アンモニア態窒素比

• [x] ph 酸性アルカリ性度合い e.g. 0.6→水素イオン濃度

• [x] cao 酸化カルシウム→交換性石灰

• [x] mgo マグネシウム→交換性苦土

• [x] k2o カリ→交換性加里

• [x] base_saturation 塩基飽和度 e.g. 0.57

• [x] cao_per_mgo CaO/MgO e.g. 0.57

• [x] mgo_per_k2o MgO/K2O e.g. 0.57

• [x] phosphorus_absorption リン吸収→リン酸吸収係数

• [x] p2o5 五酸化リン→可給態リン酸

• [x] cec 塩基置換容量

• [x] humus 腐植

• [x] bulk_density 仮比重

また参考に過去の土壌化学性診断の結果を添付します。（今検査依頼している分析機関のもの）

南 土壌化学性報告書19001775.pdf

[duri0214]

@yoshiyukiminami

• nh4+no3 の + は処理上エラーが起こる可能性が高いのでダメです（python上で加算が起こってしまうのでアルファベット、数字、アンダーバーだけです）。いったん nh4no3 にしてます。
• nh4/(nh4+no3) も、 nh4_per_nh4no3 にしておきます

[yoshiyukiminami]

nh4+no3 は　total_n でも良さそうです。 nh4/（nh4+no3）は　nh4_ratio でも

[duri0214]

@yoshiyukiminami なんどもツッコミ入ってめんどくさいなって思うかもしれませんが、命名規則ってかなり重要なんです（時間が経つほど修正が大変になってボディーブローのように効いてきます）

• total_n って total も n もかなり一般的な名前に見えますがそれは大丈夫ですか？（止めてるわけじゃなくてのちのちのことを含めて考えているだけです。わかるならそれで修正しておきます。いまは nh4no3 ）
• nh4_ratio のほうも、ratio を使うなら、ほかの項目の per がついてるやつも ratio に変えたほうがいいと思います。どちらに合わせますか？（いまは nh4_per_nh4no3 ）

[yoshiyukiminami]

total_n→total_nitrogen nh4_ratio→nh4_per_nitrogen とperで統一しましょう。

[duri0214]

• [x] nh4no3 → total_nitrogen
• [x] nh4_per_nh4no3 → nh4_per_nitrogen

[duri0214]

@yoshiyukiminami 最初のコマンド python manage.py makemigrations chemical_report を実行したときのdbまわりのエラーが思い出せないので、スクリーンショット、テキスト貼り付けをお願いします （例によって僕が再現できない系なので役に立たない可能性は高い）

[yoshiyukiminami]

manage.py(soil_analysis>hiryo)の上（#!/usr/bin/env python）を実行したときのターミナル画面スクショです。

[yoshiyukiminami]

次に同じmanage.pyのmainを実行したときのターミナル画面・スクショです。

[yoshiyukiminami]

これで良かったです？

[duri0214]

@yoshiyukiminami

• 違うコマンド実行してるっぽい
• mysqlclient が入っているか点検

[yoshiyukiminami]

明日zoomの中でやりましょう。

[yoshiyukiminami]

岡田さん

下記の件ですが、

・data_processing・・この2つのpyは使います。

• data_preprocessing.py

• tekusei_stats_cal.py

・graph・・気象庁HPから任意地点の気象データをスクレイピングするpyは使います。

• test_scraping.py

• test_scraping_csv.py

• thisyaer_data_scraping.py

• today_data_scraping.py

basic_django_graph/mysiteは勉強用に作成してもらったやつで実際に使用する仕様のpyは作成していないので、整理は可能かと思います。

南

From: YOSHITAKA OKADA @.> Sent: Sunday, August 14, 2022 12:15 PM To: yoshiyukiminami/python @.> Cc: yoshiyukiminami @.>; Mention @.> Subject: Re: [yoshiyukiminami/python] djangoアプリ再構築 (PR #42)

@yoshiyukiminami https://github.com/yoshiyukiminami

つぎの土曜日やることリスト

• この３つ使ってなさそう。整理削除しても大丈夫ですか？ https://user-images.githubusercontent.com/19668980/184520850-5d3ff70e-d54b-45b6-a073-3023f7b916e8.png
• models.py のレビューとすり合わせ
• 環境の再現を説明

— Reply to this email directly, view it on GitHub https://github.com/yoshiyukiminami/python/pull/42#issuecomment-1214275733 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AW73KUZ3GNQQGTQ65TU3D23VZBQB7ANCNFSM56OCCLEQ . You are receiving this because you were mentioned. https://github.com/notifications/beacon/AW73KUZ2F73EFY6KB6FZOBTVZBQB7A5CNFSM56OCCLE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOJBQGBFI.gif Message ID: @. @.> >