Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auto_operationの配線図とダイヤを更新 #239

Merged
merged 26 commits into from
Nov 15, 2022

Conversation

n4o847
Copy link
Collaborator

@n4o847 n4o847 commented Nov 12, 2022

Closes #232

内容

11月12日時点で Discord でやりとりされた新しい配線図にしたがって、auto_operation の内部の配線図とダイヤを書き換えました。
ダイヤグラムは docs/chofufes2022-diagram.jpg にあります。
Thanks to @Ryota-Nitto

ところでそこそこ影響の大きい変更点として、今までは各種 ID を整数で管理していたが、今回の変更にともなって文字列に変更したというのがあります。
影響する範囲を追うために型チェックを有効にして開発してみたけど、そもそもの auto_operation の型エラーが多かった……。
各種 ID について Literal 型を使って型チェックを行っているので、タイプミスはなくせていると思います。

@n4o847 n4o847 changed the title auto_operationの配線図を更新 [WIP] auto_operationの配線図を更新 Nov 12, 2022
@n4o847 n4o847 added enhancement New feature or request backend ats labels Nov 14, 2022
@github-actions github-actions bot removed the ats label Nov 14, 2022
@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 14, 2022

あ~、DiaPlanner.py も変えなきゃいけないのか

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 14, 2022

ダイヤグラムも別イシューにします

やっぱりこの PR 内でやります

@n4o847 n4o847 marked this pull request as ready for review November 14, 2022 16:19
@n4o847 n4o847 changed the title [WIP] auto_operationの配線図を更新 auto_operationの配線図を更新 Nov 14, 2022
@n4o847 n4o847 marked this pull request as draft November 14, 2022 16:23
@n4o847 n4o847 changed the title auto_operationの配線図を更新 auto_operationの配線図とダイヤを更新 Nov 14, 2022
@n4o847 n4o847 marked this pull request as ready for review November 14, 2022 18:04
@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 14, 2022

誰にレビュー頼めばいいかわからないけど、とくに @thgcMtdh かな……?

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

#262 の方で上がったのが、shinjuku_b1 上にもストップレールが必要なのではないかという

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

[main.operation_loop] t0.section: hachioji_b2, t0.mil: 172.08, t0.spd: 0.00, t1.section: chofu_b5, t1.mil: 180.27, t1.spd: 0.00
[main.operation_loop] t2.section: hashimoto_b2, t2.mil: 172.20, t2.spd: 0.00, t3.section: sakurajosui_b4, t3.mil: 172.22, t3.spd: 0.00

でデッドロック起きとる

が、そもそもの修正が大きくて、他の変更のためにも一回これをマージしておきたいので、細かいダイヤグラムの修正は別イシューでやることとします

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

あとは、#263 とけっこうコンフリクトしてるので、これを取り込んで終わり……!

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

う~ん、路線図の定義部分がフォーマットされるの見づらいので fmt: off したいという気持ち vs フォーマットしても見栄えよくなるようにうまく調整したいという気持ち

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

デッドロックを @.thgcMtdh が直してくれているので待ちます

@Azuki-bar
Copy link
Member

う~ん、路線図の定義部分がフォーマットされるの見づらいので fmt: off したいという気持ち vs フォーマットしても見栄えよくなるようにうまく調整したいという気持ち

@n4o847 これなんですけど、フォーマッタの行数制限を緩くする && コメントを末尾に書かずに行頭から始める でそれなりの見た目になりそうです。
もしくは指摘の通り除外する方法もあります。
行数制限が何故か88文字になっているのでこれを緩めてみることを別PRでまずやってみます

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

@Azuki-bar ありがとうございます、ではフォーマッタは有効にしたままで、任せます

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 15, 2022

[main.operation_loop] t0.section: hachioji_b2, t0.mil: 172.22, t0.spd: 0.00, t1.section: chofu_b5, t1.mil: 180.09, t1.spd: 0.00
[main.operation_loop] t2.section: hashimoto_b2, t2.mil: 172.14, t2.spd: 0.00, t3.section: sakurajosui_b4, t3.mil: 172.22, t3.spd: 0.00

で停止するという問題が依然起こっている( 089cebd にチェックアウトして試してみても同様だった)

この PR は一回マージして、ダイヤグラムの問題は別イシューに分割します

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

auto_operationの配線図とダイヤを書き換える
4 participants