-
Notifications
You must be signed in to change notification settings - Fork 287
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
Add PACS integration with move and query operations #644
base: master
Are you sure you want to change the base?
Conversation
I've added a progress indicator for when it's moving and in case of an exception it shows a message with the status code. hope now it gets better for user to known what is happening |
Hi @hvini! Sorry for not reviewing it yet. I wasn't able to download images from dcm4chee using c-move. I tried using Weasis to test too but I had problems. With Weasis I was able to use C-Get instead of C-Move. So I tried to use C-Get in InVesalius too. I had to make some change. Here is the diff
|
hello. I will check if dcm4chee and orthanc keeps working with c-get instead c-move and also will test the weasis tool. if i dont find any problem i will integrate your changes on code. which problem you had with dcm4chee? maybe i can help |
Hi @hvini. I have some permission problems with dcm4chee. |
hello, i tested the changes and confirmed that works for both orthanc and dcm4chee, also, with these changes there is no need anymore to configure the aet and transfer capability on dcm4chee, so, now the user needs only to upload his files on server and then will be able to query and load on invesalius. server credentials are: 0.0.0.0, 4242, ORTHANC there is 2 docker files on root that allows to start orthanc and dcm4chee server, the command to execute the files is: |
What?
This PR adds the PACS integration with C-echo, C-query and C-move operations
Why?
These changes allows users to:
How?
Since gdcm is experiencing a problem that causes the application to quit, PyNetDicom was used to implement the dimse operations
UI Panels was built using python wx widget library
Testing?
I've added a docker-compose service to easily start a local PACS server (ORTHANC) and then test the functionalities.
After install docker and docker compose, run the service with
docker-compose up
command.orthanc.json
file, in root, allows to configure the PACS server. dicommodalities refers to the servers that has permission to make requests on PACS. on c-move operation a local server starts (can be configured on application) with ip 0.0.0.0 and port 11120. since docker cant access 0.0.0.0 or 127.0.0.1, the local ip address should be added on orthanc.json, so in this way, the orthanc server starts to allows incoming requests from the local server.see orthanc documentation for more information about config file https://book.orthanc-server.com/users/configuration.html.
Screenshots
Storage server configuration into preferences
Retrieve DICOM FROM PACS panel
Patient query operation
Moved files