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

Orthanc dicom server #212

Merged
merged 5 commits into from
Sep 18, 2023
Merged

Orthanc dicom server #212

merged 5 commits into from
Sep 18, 2023

Conversation

CGDogan
Copy link
Contributor

@CGDogan CGDogan commented Sep 18, 2023

I'm not touching keycloak and pathdb, because they seem to be related to permission handling and the dicom server needs a completely different auth setup.

caMicroscope.yml Outdated Show resolved Hide resolved
@CGDogan
Copy link
Contributor Author

CGDogan commented Sep 18, 2023

The default DICOM configuration is to allow everything. Should I mention it in the README?

@CGDogan
Copy link
Contributor Author

CGDogan commented Sep 18, 2023

Sorry @birm actually I remembered that to test Orthanc, you would need to change slideloader and caracal to point to my pull requests camicroscope/SlideLoader#84 camicroscope/Caracal#163 https://github.com/cgdogan/caracal.git#fs_update https://github.com/cgdogan/slideloader.git#dicom

then DICOM explorer will be at http://127.0.0.1:8042/app/explorer.html

(after camicroscope/caMicroscope#656 , there will be a button to it)

Copy link
Member

@birm birm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

end to end test looks good. It'll be nice when we can collapse the file information to avoid having such a long table when using dicoms.

@birm birm merged commit 058cc75 into camicroscope:develop Sep 18, 2023
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

Successfully merging this pull request may close these issues.

2 participants