-
Notifications
You must be signed in to change notification settings - Fork 205
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
自動ビルド: コアを0.10.preview.3に更新, #286 の動作修正 #287
Conversation
Pull Request Test Coverage Report for Build 1696714473
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです! Draft外れてると思ってました...
あ、Readyにしてましたが、#286 関連のバグがあったので、直すためにDraftに戻しました! |
修正、ありがとうございます! |
環境変数 環境変数を指定した場合に、型変換が抜けていてコアを初期化できなかったので修正しました。 環境変数に空文字列が設定されていた場合に数値として不正とみなされてしまうので、Noneとして扱うために
@AsPulse cpu_num_threadsはNoneになるので、デフォルト動作(50%)です! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!(バグ修正もありがとうございました!)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
マージ可能ですが、@y-chanの再レビューがまだなので一応待っておきます。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
マージしますね...!
こちらのPRを反映したエンジンをビルドし始めてみました! |
Dockerイメージがあるとありがたいです…! |
|
内容
#286 がマージされたので、自動ビルドに使用するコアをCPUスレッド数を指定する機能が追加された0.10.preview.3に更新して、動作するようにします(忘れてました)。
0.10.preview.2でmetasのスタイル順が入れ替わった(あまあまが先頭に来ていたのを修正)のも反映されます。
関連 Issue
ref #286
スクリーンショット・動画など
その他