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

[Graphics] [Sample] UltraHDR Image compression #117

Merged
merged 3 commits into from
Dec 4, 2023

Conversation

MayuriKhinvasara
Copy link
Contributor

This CL contains sample code for compressing an UltraHDR Images (on API 34 and above) while still maintaining quality of the ultrahdr image

How Has This Been Tested?
Test Configuration #1 - Pixel 7 Pro API 34 Beta (Android 14 Beta)

Checklist
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas I have made corresponding changes to the documentation

Demo
The demo shows how to compressing an ultrahdr image and the difference between both.

Screenshots
ultrahdr_compression_sample

This CL contains sample code for compressing an UltraHDR Images (on API 34 and above) while still maintaining  quality

How Has This Been Tested?
Test Configuration #1 - Pixel 7 Pro API 34 Beta (Android 14 Beta)

Checklist
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation

Demo
The demo shows how to compressing an ultrahdr image and the difference between both.

Screenshots
@MayuriKhinvasara MayuriKhinvasara added the new sample This denotes the creation of a new sample in the platform samples app label Oct 23, 2023
@MayuriKhinvasara MayuriKhinvasara self-assigned this Oct 23, 2023
Copy link
Contributor

@madebymozart madebymozart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good, just a few comments on spacing and why certain things we're removed

)

class CompressingUltraHDRImages : Fragment() {

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is there all these random spaces? Can we clean this up just a little to not have them?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack. Removed.

android:id="@+id/color_mode_controls"
android:layout_width="match_parent"
android:layout_height="wrap_content" />

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is not needed, can you remove it outright?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack. Removed

@@ -25,7 +27,7 @@ Demonstrates how to implement data access auditing for your app to identify
- [Displaying UltraHDR](graphics/ultrahdr/src/main/java/com/example/platform/graphics/ultrahdr/display/DisplayingUltraHDR.kt):
This sample demonstrates displaying an UltraHDR image.
- [Displaying UltraHDR (3P Libraries)](graphics/ultrahdr/src/main/java/com/example/platform/graphics/ultrahdr/display/DisplayingUltraHDRUsing3PLibrary.kt):
This sample demonstrates using the various popular image loading libraries to display Ultra HDR images
This sample demonstrates using the various popular image loading library to
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack. was edited by mistake. Restored.

This CL contains sample code for compressing an UltraHDR Images (on API 34 and above) while still maintaining  quality

How Has This Been Tested?
Test Configuration #1 - Pixel 7 Pro API 34 Beta (Android 14 Beta)

Checklist
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation

Demo
The demo shows how to compressing an ultrahdr image and the difference between both.

Screenshots
This CL contains sample code for compressing an UltraHDR Images (on API 34 and above) while still maintaining  quality

How Has This Been Tested?
Test Configuration #1 - Pixel 7 Pro API 34 Beta (Android 14 Beta)

Checklist
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation

Demo
The demo shows how to compressing an ultrahdr image and the difference between both.

Screenshots
@MayuriKhinvasara MayuriKhinvasara merged commit 2b8d309 into main Dec 4, 2023
4 checks passed
@yrezgui yrezgui deleted the ultrahdr_compression branch December 7, 2023 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new sample This denotes the creation of a new sample in the platform samples app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants