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

Add FHIR Terminology Service #2

Open
zgypa opened this issue Dec 17, 2024 · 2 comments
Open

Add FHIR Terminology Service #2

zgypa opened this issue Dec 17, 2024 · 2 comments

Comments

@zgypa
Copy link
Member

zgypa commented Dec 17, 2024

The Plan is to transform this into a light-weight terminology service. So possibly still have the static classes, which get imported, have the UI from medoco/tops-dicom, possibly separate the terminology part from the DICOM part, so that tops-dicom only has the DICOM and would look up the ConceptMaps here to find out what to use to encode the MWL.

The service should therefore have a way to also associate the DICOM object with each other, essentially replicating the existing logic that is in tops-dicom.

The end user would then have a single entry point, to configure all mappings for the entire network. For example, query this service with a procedure_id of system tops and ask the equivalent DICOM code, which should return a code that can be used for a Requested Procedure. Then query again to get procedure steps, which would return a ValueSet or procedure Steps, and so on.

I think we would use CodeSystems to list all codes per context, so there would be a DICOM Requested Procedure CodeSystem, with a list of all availble codes in there, then another for Scheduled Procedure Steps, and so on.

@zgypa
Copy link
Member Author

zgypa commented Dec 18, 2024

I have created the branch and started importing stuff from tops-dicom. However nothing is tested and working yet. Consider using fastapi for fhir?

@zgypa
Copy link
Member Author

zgypa commented Dec 24, 2024

Started creating unittest functions to test the most important and used /ValueSet/$expand endpoint. Other tests generated by chatgpt, but not useful, i commented them out. I had to create a property for the url of ValueSet becauese of pydantic.

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

When branches are created from issues, their pull requests are automatically linked.

1 participant