Efferent-Health / fo-dicom.Codecs

Cross-platform Dicom native codecs for fo-dicom
Other
65 stars 23 forks source link

Test host process crashing from time to time, seems to be codec related #9

Open amoerie opened 4 years ago

amoerie commented 4 years ago

Related issue in fo-dicom: https://github.com/fo-dicom/fo-dicom/issues/1072

@jaime-olivares @cbeltran1306

I would love to get a little assistance with this problem, because it's hindering unrelated work (CI appears red while in fact there are no new bugs / issues ) and I'm a little on unknown terrain. Any way I could get you guys more output / detailed logging so this can be diagnosed?

In fact, the problem seems to be related to the codecs being called in parallel threads. Does this library not support multithreading? The README seems to suggest it doesn't:

image

@gofal Is this not a big problem? I think in our use cases this would be a very breaking change.

amoerie commented 3 years ago

Hi guys, can you provide any input here? @jaime-olivares @cbeltran1306 Do we know

If you need help with anything, we might be able to work something out.

jaime-olivares commented 3 years ago

Hi @amoerie Indeed parallel is not supported yet. We have approached a big company that may justify the project. Will have news soon.

amoerie commented 3 years ago

@jaime-olivares Thanks for getting back to me. Looking forward to more updates!