Andrey Fedorov

Results 803 comments of Andrey Fedorov

Another consideration for adding support of measurements is its impact on the memory footprint - even without supporting measurements, we already have some instances in IDC where we run out...

> Is there still room / ideas to make the code retrieving and managing the annotations more efficient or are we already out of options there? I don't think we...

Thank you for the report! You are welcome to submit a fix to this issue!

I would move the menu icon in the bottom left corner to the top - I think this is more conventional. I find the letters PSC quite confusing - I...

Yes, makes sense. Maybe something like the button they use in video players to show full screen?

@CPBridge it is the same collection we discussed earlier in https://github.com/ImagingDataCommons/highdicom/issues/344#issuecomment-2810500945. Yes, definitely looks like a problem with the data, and very likely with the tool, but it will be...

> I specified a schema for the underlying SQLite lookup tables that highdicom constructs when you load a segmentation @CPBridge what is the schema that you use? Where is StudiesContainingOtherReferencedInstancesSequence...

It is great that the conversion can be disabled and the original data can still be accessible!

@jamesobutler thank you so much for the contribution! I asked Copilot about the failing test, and this was the (most relevant, I think, part of the) response: > This is...

> I would favour requiring pydicom>=3.0.0 and removing the import hackery I was thinking the same, but wanted to hear from Chris or Steve, since they have a lot more...