alkee-skia / mars-processor

issue tracker for MARS processor
1 stars 0 forks source link

segmentation mask VR 설정을 사용자가 수정할 수 없게 #30

Open alkee-skia opened 3 years ago

alkee-skia commented 3 years ago

현재 사용자가 VR 설정을 통해 임의로 값을 수정할 수 있는 상태인데, 사용자가 사용할 수 없도록 해야합니다. 색상지정등이 필요한 사용자는 Segmentation 의 UI 를 통해 색을 변경해야합니다.

VR setting color Segmentation UI
image image

또한, 실제 데이터의 intensity 영역을 사용하고있는 것으로 보여, 부작용을 최소화 할 수 있도록 범위를 최소화 해야할 것으로 보입니다.

(제안)

jhpark-rs commented 3 years ago
  1. dense bone이 3,000 HU 근처의 값을 가질 수 있음을 확인하였습니다. 이에 lesion 할당 값을 +500 하였습니다.
  2. 사용자 변경 불가능하게 한다고 하면 현재 톱니 모양의 아이콘 color setting button을 비활성화 해야 합니다. custom coloring 기능 차단이 원하는 방법인지 확인 바랍니다.
alkee-skia commented 3 years ago

custom coloring 기능의 차단이 원하는 방법이 아니라, 유저가 잘못된 조작으로 잘못된 결과를 초래하는 여지를 막고자 하는 것이 목적입니다.

mask 가 HU 값을 가져야만 한다면 각각 6,000 이상으로 narrow 한 range 를 갖도록 하고, custom coloring 에서는 6,000 미만의 값만 표시할수는 없나요? (6,000 은 예시. 충분히 큰값)

jhpark-rs commented 3 years ago

사용자 오조작 막을 좋은 방법 있으면 추천 바랍니다. 6,000 기준에 장점이 있나요? 3,500부터 7,000까지 쓰는 이유는 현재 구현상 coloring range가 -1024 ~ 7168 이고, lesion간 500 HU 간격을 두어 interpolation에 의한 색번짐 현상을 줄이기 위해서입니다.

alkee-skia commented 3 years ago

사용자 오조작 막을 좋은 방법 있으면 추천 바랍니다.

추천드린 방법이 custom coloring UI 에 HU 범위를 mask 영역보다 작은 영역까지만 표시하는 것(실제 값은 사용하더라도 UI 에 표시 상한을 지정)인데, 이 방법이 어렵다고 하시면 조금 더 고민해보도록하겠습니다. 다만, 가장 간단한 해결법인 것 같아 제안 드린 것이기 때문에 구체적인 구현내용을 모르는 저희로써는 만족스러운 방법을 추천할 수 있을지 모르겠습니다.

6,000 기준에 장점이 있나요?

6,000 수치가 장점을 갖는 것이 아니라 위 제안드린 방법(UI 범위 제한)에서 데이터 영역이 기존 제공하던 영역에 비추어 그나마 덜 훼손될 것 같아 제시한 수치일 뿐입니다.

jhpark-rs commented 3 years ago

이해했습니다. 사용자가 색상을 변경하게 하거나, 그렇지 않을 경우 좀 더 시인성 높은 색상으로 먼저 구성한 다음에 제안 방안 포함, 오조작 막는 방법을 구현해보겠습니다. 일단 우선순위는 낮은 것으로 알겠습니다.