Skip to content

コマンドモジュールを再読込するボタンを追加#11

Merged
KawaSwitch merged 1 commit intoKawaSwitch:masterfrom
vyPeony:Add-reload-command-button
Jan 25, 2020
Merged

コマンドモジュールを再読込するボタンを追加#11
KawaSwitch merged 1 commit intoKawaSwitch:masterfrom
vyPeony:Add-reload-command-button

Conversation

@vyPeony
Copy link
Copy Markdown
Contributor

@vyPeony vyPeony commented Jan 25, 2020

概要

MCU, Python, Utility のモジュールをアプリを落とさずに再読み込みするボタンを追加します。

変更点

image
※表示されているコマンドは検証中のもので無関係です。本PRには含まれません。

使い方

ボタンを押すだけです。

影響範囲

なし。

補足事項

  • Python モジュールは実行中でも再読み込みできるようですが、予期せぬエラーの可能性はあると思われます(手元では未発生)。
    一旦停止してから再読み込みしたほうが安全と思われます。
  • Python は不慣れなのでよりよい実装方法があればご指摘ください。

コマンドだけリロード用のボタンが存在しなかったので追加
スクリプトの調整などで使いたいことはあると思われる
@KawaSwitch KawaSwitch added the enhancement New feature or request label Jan 25, 2020
@KawaSwitch
Copy link
Copy Markdown
Owner

お疲れ様です。便利な機能のプルリクエスト大変感謝いたします。
こちらの環境でも動作を確認いたしましたのでマージさせて頂きます。

主に試したのは次の3点です。

  • コマンド実行中の再読み込み
    コマンドを実行しているスレッドが独立しているため特に問題はなさそうです。こちらでも試している限りではエラーは出ませんでした。

  • 再読み込み時に現在表示しているコマンドがcommandsから削除されている場合
    一致判定処理が飛ばされ0番目が選ばれるので問題なし。

  • 再読み込み時にcommandsが空
    例外となるが考慮しなくてもいいですよね。

Python は不慣れなのでよりよい実装方法があればご指摘ください。

私自身もpython(特にtkinterによるGUI実装)は経験が浅くあまりレビューできることはありませんが, 特に問題ないかと思います。

@KawaSwitch KawaSwitch merged commit a94d565 into KawaSwitch:master Jan 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants