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.

Sign up for GitHub

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

Jump to bottom

auto_operationのFlaskやWebコントローラーを削除する #233

Open
n4o847 opened this issue Nov 12, 2022 · 3 comments
Open

auto_operationのFlaskやWebコントローラーを削除する #233

n4o847 opened this issue Nov 12, 2022 · 3 comments
Assignees
@n4o847 @thgcMtdh
Labels
ats backend enhancement New feature or request

Comments

@n4o847
Copy link
Collaborator

n4o847 commented Nov 12, 2022

内容

  • auto_opeartion はもともと、Flask サーバーを立てて、Web 上のコントローラーから運転体験ができるというものだった
  • backend/auto_operation の static や template 以下にはこれらのデータが含まれているが、今回は不要なので削除する

Pros 良くなる点

  • Web コントローラー用にあった不要なファイル・処理がなくなる

Cons 悪くなる点

実現方法

その他
@n4o847 n4o847 added enhancement New feature or request backend ats labels Nov 12, 2022
@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 16, 2022

Flask も要らないのでは？

@n4o847 n4o847 changed the title auto_operationのWebコントローラーを削除する auto_operationのFlaskやWebコントローラーを削除する Nov 16, 2022
@n4o847 n4o847 mentioned this issue Nov 16, 2022
Merged
@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 16, 2022 •
edited
Loading

あ～、でも WebSocket 無いと debug/auto_operation_visualizer が使えなくなっちゃうのか

とはいえ閉塞情報も internal に流れてくるっぽいし、要らなくなる気もする

@n4o847
Copy link
Collaborator Author

n4o847 commented Nov 16, 2022 •
edited
Loading

Flask も残してポートは設定できるようにするなど？

#278 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@n4o847 n4o847

@thgcMtdh thgcMtdh

Labels
ats backend enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@n4o847 @thgcMtdh