Davide Punzo

Results 116 comments of Davide Punzo

@GitanjaliChhetri leaving this to you, but it should be ready, and just merge it

@pieper some tests are failing, but they not related to this PR, and they fail on master branch too. It looks like first detect was at commit https://github.com/dcmjs-org/dcmjs/commit/42a7ca7d007ef12a0f287d9b23771c1516fab6ab

> @Punzo thanks for digging into this +1 > > Do we know what the segmentations are supposed to look like for this dataset? From what I see the first...

> > Maybe indeed we should shelve support of those collections until we have FRACTIONAL support implemented. Up to you, but even with proper support, the shape of the segmentation...

> > Maybe indeed we should shelve support of those collections. > > Given this discussion that's what I would vote for. Ok then for the second segmentation, i will...

> > > Maybe indeed we should shelve support of those collections. > > > > Given this discussion that's what I would vote for. > > Ok then for...

> > Maybe indeed we should shelve support of those collections > > Given this discussion that's what I would vote for. P.S.: if you meant that we should just...

> What do you all think? Is there any objection against replacing all the icons in Slicer with simple line drawing icons similar to Google material icons? It would be...

> I used the default: ![image](https://user-images.githubusercontent.com/7985338/220539685-5440363c-eae2-4657-b84e-2cf5e054cfcb.png) > > ![image](https://user-images.githubusercontent.com/7985338/220539305-2d35ac4f-a451-4c14-9396-b0f2a169e5a9.png) Experienced that it would be better to get the icons in 24 px for the optical size, since Qt will not...