Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plugin does not abide by "StoreDicom" : "false" configuration option #15

Open
jasperdenotter opened this issue Aug 19, 2020 · 2 comments

Comments

@jasperdenotter
Copy link

Hello,

I have recently been using this plugin to use Orthanc while saving disk space by storing all DICOM files in S3 storage, it works great! I have a question regarding the "index-only" mode on Orthanc and it's compatibility with this plugin; at the moment it looks like the plugin disregards the "StoreDicom" configuration option and always stores the entire DICOM file in S3. Would it be possible to modify the plugin to listen to this configuration option so that when "StoreDicom" is set to "false" only the JSON summary of the DICOMs is stored and not the entire file?

I tested this by setting "StoreDicom" to false, uploading some files, and then checking if I could still view them on the osimis web viewer. In this Case I was able to still see the full images were being stored. Let me know how involved it would be to make this change and if this is something you would consider implementing. I'd be happy to answer any questions you have.

Thanks!

@gmanuel-vitall
Copy link

gmanuel-vitall commented Oct 14, 2020

Hi @jasperdenotter I am trying to run the dockerfile to get this running but am getting an error statement Error libcrypto.so.1.0.0: cannot open shared object file: No such file or directory. Did you ever come across this issue and if so, how did you resolve it?

Thanks!

@bpycinski
Copy link
Contributor

Hi @jasperdenotter, thanks for being interested in Orthanc and sorry for a late response. Previously we had distinguished dicoms from jsons, this was removed with a commit 7ebd9b3 . But you can easily switch the condition to store only jsons on s3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants