SDUpdaterに対応する

[ronron-gh]

NoRiさんのｽﾀｯｸﾁｬﾝ向けSDUpdaterに対応させる。

• 現状NoRiさんのSDUpdaterはCore2向けのため、他の機種向けにビルドするときはコンパイルオプションでSDUpdater用コードを無効化できるようにする。
• 他のソフトとの設定ファイルの共通化を検討する。YAML、タカオさんのライブラリを使用。

[mongonta0716]

@NoRi-230401 で呼べないですかね？

[NoRi-230401]

今、メール案内から承諾してここにたどりつきました。

[ronron-gh]

@mongonta0716 @NoRi-230401
コメントくださったおかげで、@で候補に出てくるようになりました。

本題ですが、こちらのコミットで、まずはSDUpdater対応してみました。
a5d5013

platformio.iniの[env:m5stack-core2-sdu]でビルドしたときだけSDUpdater対応になるようにしてみました。
もっと良い方法がありましたらご遠慮なくコメントください。
サーボPIN設定はひとまずNoRiさんのservo.txt方式を取り入れましたが、これからYAML方式を検討していきたいと思います。

[NoRi-230401]

さっそく、ビルドして使用してました。
私の「動けばいい的ソフト」から汎用性を持たせることができるよう変更するには、こうゆう風にすればいいんだと感心しております。改めて取り込まれた自分のソースを見るとエラー処理がいい加減で申し訳ないです。

[mongonta0716]

SDUpdater.hとSDUpdater.cppの関数をstackchan-arduinoライブラリで持ってもいいかもしれませんね。

[NoRi-230401]

提案です。
"PROG_NAME"（lobbyで表示されるソフトの名前）を定義している場所は、

SDUpdater.cppで ２３行目
　　const String PROG_NAME = "AiStackChan2FuncCall";

(１)．これをmain.cppで定義するようすると、他のソフト毎にSDUpdater.cppの変更する必要がない。
(２)．この名前自体を設定ファイルから読取方式にするとmain.cppもソフト毎に変更する必要がないのでいいかも。

すいません、追記です。よく考えたら(２)は、SDU_lobby（）は、M5.begin直後に呼ばれるので無理そう。
結局(１)のmain.cppに埋め込むのがいいのでは。

[NoRi-230401]

今回のSDUpdaterとは関係ないのですが、前に調べていてわかったのですが、
「Error attaching servo xError attaching servo y」のエラーが出ていています。（シリアルモニタ）
main.cpp 225-230 の合否判定が逆になっているようです。
この機会に訂正お願いします。

[ronron-gh]

ご確認ありがとうございましたm(__)m

SDUpdater.hとSDUpdater.cppの関数をstackchan-arduinoライブラリで持ってもいいかもしれませんね。

確かにそうですね。十分に汎用的になったところで、stackchan-arduinoライブラリへ移動したいです。

"PROG_NAME"（lobbyで表示されるソフトの名前）を定義している場所

確かに、SDUpdater.cpp内で定義すると他のソフトで使いづらいですね。
提案ですが、このようにmain.cppでSDU_lobby()を呼ぶときに引数で渡すのはいかがでしょうか（こちらで改造し、問題なく動作しました）。

  SDU_lobby("AiStackChan2FuncCall");

「Error attaching servo xError attaching servo y」のエラーが出ていています。（シリアルモニタ）

情報ありがとうございます。修正を取り込みます。

[NoRi-230401]

SDUpdater.cppの修正いいと思います。よろしくお願いします。

[ronron-gh]

上記の修正をコミット( ed101ae )したので、YAMLのほうに着手しようと思います。
細かな方針について次のようなことを考えています。ご意見ありましたらお願いいたします。

• YAMLファイルがSDカードに無い場合は、レガシーなTXTファイルを読み込む。TXTファイルから読み込んだ場合はYAMLファイルに書き起こしてSDカードに新規保存することで、ユーザーのYAML移行を促したい。
• SC_ExConfig.yaml（アプリ毎の設定）は、SDUpdaterで複数アプリが共存する場合はアプリと同じ数のSC_ExConfig.yamlが出来るのか、一つのSC_ExConfig.yamlに複数アプリの設定を保存するのかを決めたい。以下の理由で自分的にはアプリ毎のフォルダを作った方が良い気がしていますが、他に妙案等あればお願いいたしますm(__)m
  • アプリ毎にファイルが出来る場合は、アプリ毎にファイル名を変えるか、アプリ毎のフォルダを作って保存する必要がある。（そもそもYAMLファイルに限らずSDカード直下にいろいろなアプリが作ったファイルが混在すると使い勝手が悪いので、アプリ毎のフォルダはあった方がよいのではないか）
  • 一つのファイルにすればスッキリするが、あるアプリが間違ってファイルを壊すと他のアプリにも影響する。

[mongonta0716]

アプリ側で決められるように、SC_ExConfig.yamlという名前を変えられるようにしましょう。SC_BasicConfig.yamlとSC_SecConfig.xmlは名前を固定してもいいような気もしますね。

StackchanSystemConfigのコンストラクタでSC_BasicConfig.yamlではなく、SC_ExConfig.yamlの名前を指定するようにしましょうか。

[mongonta0716]

変更しました。

mongonta0716/stackchan-arduino@8456089

[NoRi-230401]

・私も二人の意見とほぼ同じなのですが、縛りをきつくすると窮屈さを感じるのでSC_BasicConfig.yamlとSC_SecConfig.xmlとライブラリを使えば、そのまま便利に使える共通部品として便利になる。それ以外は、設定ファイルも各アプリで名前を変えて自由に使えるようになるといいと思います。

・レガシーTXTをサポートは嬉しいです。
ただ、心配なのが、SD書込みは失敗がありそうなので、その時にファイルが破損しないかが気になりました。
自分の経験だと、PCを使用して問題ないSDへの書込みが、M5StackCore2直で書込みが失敗しファイルが破損することがあるので（SDの相性？？）書込みや止めた方が無難と思っています。読み込みは比較的大丈夫だと思います。
もしかして、私の固有の問題かもしれないのです。

訂正です。よく考えて見たら、もともと存在なかったファイル作成するんだから失敗して大丈夫ですね。
すいません、よけいな懸念ですね。

・アプリ毎のフォルダもあった方が、アプリが他の影響うけにくいので作りやすいですね。
アプリ毎フォルダ下にアプリ専用の設定ファイルを持ってくるのがやはり自然ですね。
共通部分だけの設定ファイルの位置と名前とを決めるのかなあ。

[mongonta0716]

書き込みと相性は確かに怪しい部分があります。特にCoreS3、、、KIOXIAとSandisk以外は信用できませんね。。。
あまり長いフルパスファイル名も使えないので、、、下記のようにしてしまうとか、、、
/yaml/SC/Basic.yaml（読み込み専用）
/yaml/SC/Secret.yaml
/yaml/<フォルダも含めてAPPで自由>.yaml

フォルダも自由ぐらいで収めておいてもいいかも。

[ronron-gh]

ご意見ありがとうございました。書き込みのトラブルは確かに心配ですね。新規作成のみとはいえ、内容が中途半端なファイルが出来て次回起動時に読み込みエラーという可能性もあるので、ひとまず書き込みは無しで進めたいと思います（単に、YAMLが無ければレガシーTXTを読み込む、に留める）。

ルールを決め過ぎてアプリ開発に参入しづらくなるのも確かに避けたいですね。「stackchan-arduinoライブラリを使えば簡単にベース部分を作れる」というコンセプトも大事にしたいですね。それを踏まえると、以下のような説明のしかたが良いでしょうか。

• stackchan-arduinoライブラリを使えば以下の設定ファイルから簡単に共通的な設定を読み込める。（あくまで推奨）
  • /yaml/SC_Basic.yaml ・・・共通的な設定
  • /yaml/SC_Secret.yaml ・・・共通的な設定（個人情報）
• アプリ固有の設定ファイルや一時ファイルは以下のアプリ固有フォルダを作って保存することを推奨する。
  • /app/(アプリ名)

@mongonta0716
アプリ固有のファイルはyamlに限らないと思ったため、フォルダ名を"app"にしてみました。