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

UL SCP API #140

Open
BastienVigneron opened this issue Jul 11, 2021 · 3 comments
Open

UL SCP API #140

BastienVigneron opened this issue Jul 11, 2021 · 3 comments

Comments

@BastienVigneron
Copy link
Contributor

Hello,

Thank you for this gorgeous lib.
Did you have any plan to implement SCP part of upper layer protocol ?
Something like a set of Traits the user ca implement for StoreSCP, MoveSCP, FindSCP, ...

@Enet4
Copy link
Owner

Enet4 commented Jul 12, 2021

Hello! Thank you for your interest in DICOM-rs.

Yes, it makes sense to continue expanding towards higher-level abstractions, for both SCU's and SCP's.
While I am not entirely sure how traits would work here, the original vision after dicom-ul emerged was to conceive something like a set of state machines, to cover the boilerplate of handling certain requests according to the standard. This will hopefully also be expanded to support asynchronous constructs. For now, implementing these services can be done on top of the dicom::ul::association abstraction, which already has an API for server-sided associations.

On my personal priority list, lazy DICOM object loading and finalizing the baseline for the pixel data abstraction ought to be made available in the next major release. But should anyone be interested in pushing this initiative forward, I would be more than glad to help.

@BastienVigneron
Copy link
Contributor Author

BastienVigneron commented Jul 12, 2021

Thank you for your response.
I will take a look to association API.

Agree for lazy loading priorities, this will be a big improvement.

@Enet4 Enet4 changed the title UL SCP UL SCP API Sep 30, 2023
@Enet4
Copy link
Owner

Enet4 commented Sep 30, 2023

Related: #352

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

No branches or pull requests

2 participants