Skip to content

Morphological image processing#209

Merged
smikitky merged 6 commits into
masterfrom
morphological-image-processing
Dec 7, 2021
Merged

Morphological image processing#209
smikitky merged 6 commits into
masterfrom
morphological-image-processing

Conversation

@ttakenaga
Copy link
Copy Markdown
Contributor

No description provided.

@ttakenaga ttakenaga force-pushed the morphological-image-processing branch from 5600f99 to bd1d037 Compare December 5, 2021 12:10
@ttakenaga ttakenaga force-pushed the morphological-image-processing branch from bd1d037 to 4c8046e Compare December 6, 2021 00:26
Copy link
Copy Markdown
Member

@smikitky smikitky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

erosion/dilatation用のダイアログのデザインを調整しました。

  • CIRCUSにはダークモードがあるので、決め打ちで色を書くのはできるだけ避けてください。どうしてもやる場合はダークモードとライトモードの両方でデザインを確認してください。
  • classnames を使えるようになってください。
  • いまどき float を使ってグリッドをレイアウトするのは止めましょう…。
  • コンポーネントへの型付けのしかたを他のコードベースと統一してください。

以下の修正をお願いします。

  • structure に型名を付けてください。
  • erosion / dilatation のシグネチャは共通だと思うのでそれについても型を付けて、同じシグネチャで扱えるコードだというのを明示しましょう。
  • performLabelCreatingVoxelProcessing.ts に、大部分が同じような長いコードのコピペが現れています。似たような処理の流れを可能な限り統一的に扱うために書いたコードなのに、それ自体をコピペで増やしていたのでは本末転倒です。シグネチャを工夫したら統一できませんか?

nSlices: number;
};

export type BasicMorphologicalOperations = (
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

複数形なのはおかしいので Operation で。

@ttakenaga ttakenaga force-pushed the morphological-image-processing branch from 6d9adea to 5230272 Compare December 7, 2021 05:55
@smikitky smikitky merged commit bb048a6 into master Dec 7, 2021
@smikitky smikitky deleted the morphological-image-processing branch December 7, 2021 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants