Enet4 / dicom-rs

Rust implementation of the DICOM standard
https://dicom-rs.github.io
Apache License 2.0
414 stars 82 forks source link

Encoding to JPEG transfer syntaxes #303

Closed Almeida-a closed 1 year ago

Almeida-a commented 1 year ago

This is a first version of an implementation regarding issue #286.

It it not tested yet, however, any suggestion regarding that, or other mistakes, would be appreciated!


Depends on #361

Almeida-a commented 1 year ago

Right now I just need to finish the unit tests, but since I am encountering some errors and I am still a beginner in rust, I'm taking longer to complete the rest of this pr (sorry for that); but if at any point there is a need to more quickly finish this, I can commit my work up until this point.

Enet4 commented 1 year ago

Hey @Almeida-a, don't worry. As I intend to change the pixel data adapter API soon, it might be worth stalling this a bit more while I set up the new design, then we can work together on migrating the implementations. I hope that this works for you.

Enet4 commented 1 year ago

Update: #361 is what I have in mind. When you have the time, please rebase this branch against it and try to adjust the implementation. Feel free to ask if you need any help.

Enet4 commented 1 year ago

I took the liberty of rebasing this PR on top of #361 to start trying things out. I also took care of error handling in the encoder. Next I believe that this would call for a test suite to cover converting some pixel data to JPEG and back.

Enet4 commented 1 year ago

New stuff:

I found a few more things worth addressing at #361, but the changes to be made here should be minimal. I will take care of rebasing afterwards.

Enet4 commented 1 year ago

@Almeida-a Can you have a look and see if the new changes make sense to you? It should hopefully be ready to merge soon.

Almeida-a commented 1 year ago

On it!

Almeida-a commented 1 year ago

Looks good, thank you for your assistance!

Enet4 commented 1 year ago

Great! Thank you for the initiative and commitment, @Almeida-a!