Auto button for loader `rotation_angles` parameter sets value incorrectly
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 75/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- frontend
調査の方向性
frontend/src/contexts/LoaderContext.tsx の rotation_angles 型から始め、次に Loader コンポーネントの auto トグルを追跡します。設定が data_path の下に "auto" をネストするのではなく、rotation_angles: "auto" を使用するように、パラメータの形状とトグルの動作を更新します。その結果のワークフローが期待される設定を受け取ることを確認します。
索引モデルが issue の本文から書いたものです。
説明
The typescript type used to represent the rotation_angles parameter of the loader is not quite correct regarding where the auto should be placed. The type has this: https://github.com/DiamondLightSource/TomoHub/blob/1bc40efd239fd37e575970a062e9dea8bea53f2f/frontend/src/contexts/LoaderContext.tsx#L15-L21
but the auto should be for the entire rotation_angles field, not the data_path subfield.
Ie, the type should look something like this instead:
rotation_angles: {
data_path?: string
user_defined?: {
start_angle: number;
stop_angle: number;
angles_total: number;
} | "auto" | null; // Allow null and 'auto';
A consequence of the incorrect type is that when the auto toggle button in the Loader component is toggled on, this causes the rotation_angles parameter config to be this:
rotation_angles: {
data_path: "auto"
}
which is incorrect, it should be:
rotation_angles: "auto"
This ultimately leads to a workflow triggered with the auto button toggled on to fail, because httomo is given an incorrect configuration of the rotation_angles parameter.
- 主要言語
- TypeScript
- スター
- 2
- フォーク
- 1
- 平均マージ
- 17時間 16分
- マージ済み PR(30日)
- 44
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
DiamondLightSource/ImagingHub のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
DiamondLightSource/ImagingHub#187 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
DiamondLightSource/ImagingHub#104 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
DiamondLightSource/ImagingHub#89 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
DiamondLightSource/ImagingHub#44 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 55/100
DiamondLightSource/ImagingHub#189 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
DiamondLightSource/ImagingHub の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
juice-shop/juice-shop#3686 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
dsh-market/dsh-market#877 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
ScottKirvan/GOMAC#61 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
stellar/stellar-dev-skill#151 ·
メンテナーはふだん 1 日以内に返信