search

camicroscope / caMicroscope

Digital pathology image viewer with support for human/machine generated annotations and markups.
BSD 3-Clause "New" or "Revised" License
247 stars 286 forks source link

create visualization graph #1011

Closed YukaUU closed 1 month ago

YukaUU commented 1 month ago

Add visualization panel to viewer page.

Summary

Click the icon to the left of the toolbar's tutorial to open the side menu on the right side. Then, you can see a graph visualizing the annotation information It shows the relationship between human:annotation vs zooming. all

Motivation

To visualize pathologist annotation information

Testing

The graph and the results with collection:Mark. Annotation:0 d0 d0_log Annotation:1 d1 d1_log Annotation:9 d9 d9_log

YukaUU commented 1 month ago

Can we call this new field something a little more reasable, like viewerState?

OK! I change new field name.

Maybe you want to save it somewhere inside, so that each drawn object has its own x,y,zoom, even if they're grouped together? I noted some issues when testing this in #1013

That's right. I will fix it.

YukaUU commented 1 month ago

I changed new field name from states to viewerStates.https://github.com/camicroscope/caMicroscope/pull/1011/commits/29e4a084f00cffc164e5592cca3fbaba0786f6a4

Test

  1. Show x, y and zooming in pop panel image

2.log (send data to DB) change_states_viewerStates

3.log:DB collection:Mark chabge_states_viewerStates

YukaUU commented 1 month ago

Maybe you want to save it somewhere inside, so that each drawn object has its own x,y,zoom, even if they're grouped together? I noted some issues when testing this in https://github.com/camicroscope/caMicroscope/issues/1013

Not all annotations saved have (accurate) info saved #1013https://github.com/camicroscope/caMicroscope/issues/1013#issue-2414149542

I fixed it!

Test

  1. Annotation: Multi mode

    Annotation: Multi mode

  2. log (send data to DB) image

3.log:DB collection:Mark image

  1. Check pop up panel image image image image image
YukaUU commented 1 month ago

Delete viewerStates Past PR did not support Multi Mode.　https://github.com/camicroscope/caMicroscope/pull/966 I fixed it https://github.com/camicroscope/caMicroscope/pull/1011#issuecomment-2236869813 This commit was also deleted.https://github.com/camicroscope/caMicroscope/pull/1011#issuecomment-2236230522

Test

  1. Annotation: Single mode image log:DB collection:Mark image

2.Annotation: Multi mode image log:DB collection:Mark image

YukaUU commented 1 month ago

@birm Refactored to remove unnecessary code, etc I resolved all comments and issue. （https://github.com/camicroscope/caMicroscope/pull/1011#pullrequestreview-2183556879 , https://github.com/camicroscope/caMicroscope/issues/1013) Please check it

Test

Visualization Graph image log (send data to DB) image log:DB collection:Mark image Pop up panel image image image image image

Other(It is not related to this PR.) All fixes in this PR are done, but I found a bug in the Pop Panel display, added to the issue When two or more annotations overlap, only one of them is displayed when trying to display the PoP Panel. #1014