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 Option to Limit File Size? #2

Closed
YoungCatChen opened this issue Feb 1, 2022 · 2 comments
Closed

Add Option to Limit File Size? #2

YoungCatChen opened this issue Feb 1, 2022 · 2 comments

Comments

@YoungCatChen
Copy link
Contributor

Hello Manu,

You may not know me, but I have been very grateful for this Lightroom Plugin that you built. I saved gigs of storages by re-export my library to HEIC, instead of JPG, while keeping the same quality. Thank you!

While I'm using this plugin, I found that I missed the "Limit File Size To" feature that Lightroom has built for JPG output. I wish your plugin could have it too!

You may or may not have the bandwidth for this feature... If you don't, I'm happy to help and to contribute to your plugin. A naive approach that I have to let the ExportHEICCommand.swift file call ctx.writeHEIF(10)Representation() multiple times, with different quality, to find the best approximation. If you are open to community contribution, do you feel good about it?

If you are happy about me contributing to your project, it will be great if you write down the instructions that someone can use to set up the environment. For example, clone your code, install certain SDK, compile, set this or that settings, check debug logs, etc. They will help a lot people like me who want to ramp up with this project.

Thank you in advance. Look forward to your reply!

YoungCat

@milch
Copy link
Owner

milch commented May 4, 2022

Hey, apologies for the delay in response, I wasn't getting email notifications for this repo from GitHub. I'll take a look at your PR

@milch
Copy link
Owner

milch commented May 4, 2022

I merged your CR, thanks again for your contribution. Happy that this is helping someone out there.

@milch milch closed this as completed May 4, 2022
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

No branches or pull requests

2 participants