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

Implementation of Softcopy Presentation States #139

Merged
merged 187 commits into from
May 16, 2022

Conversation

CPBridge
Copy link
Collaborator

Implementation of GSPS. Focus is on text and graphic annotations, other optional functionality such as shutters, masks, crops, and transformations is omitted, but could be added later.

Tested in Osirix viewer.

@CPBridge CPBridge added the enhancement New feature or request label Nov 19, 2021
@CPBridge
Copy link
Collaborator Author

Can't seem to add @seandoyle as a reviewer(?) but would be great if he could take a look also!

@hackermd
Copy link
Collaborator

Can't seem to add @seandoyle as a reviewer(?) but would be great if he could take a look also!

We will need to add him to the team. However, he should still be able to just a review.

@hackermd
Copy link
Collaborator

Nice @CPBridge! Looks great at first glance. I will take a closer look and review.

It would be great if we could add support for the LUTs of the grayscale transformation:

  • Modality LUT
  • VOI LUT
  • Presentation LUT
  • Palette Color Lookup Table

@CPBridge
Copy link
Collaborator Author

Sure, I'll add in the LUT parts

@CPBridge CPBridge marked this pull request as draft November 20, 2021 14:14
@CPBridge
Copy link
Collaborator Author

@hackermd so are we good to go on this now?

@hackermd hackermd merged commit 3fb6d31 into master May 16, 2022
@hackermd hackermd deleted the feature/presentation_state branch May 16, 2022 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants