-
Notifications
You must be signed in to change notification settings - Fork 804
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
About culling down ACES versions. #13
Comments
I'm all for getting just the current ACES config in the master repo. I would like to maintain the older versions by keeping a separate branch called "legacy". This should substantially decrease the repo size and make it quite clear to users what they should grab. |
I could be wrong but without re-writing history I'm not sure the gains will be significant as you will still have the files in a blob within git tree. |
In my quick tests I'm seeing a download size reduction from 877.1 MB to 347 MB |
Download of the zip will decrease, but the repo size (what you get if you |
I'm all for removing the old ACES configs, and for that matter, even putting the extant ones in git LFS rather than in the main repo. |
While you are at it there still isn’t even a tagged released for aces
1.0.0-1.0.3 on either the Sony or the AMPAS repo. The last one was a
1.0_r2. We still grab it from HP’s repo because he has all versions
properly tagged and released there. It would be good to have these
properly setup.
…On Fri, Feb 16, 2018 at 19:52 Thomas Mansencal ***@***.***> wrote:
I would be keen on discussing about potentially removing the various ACES
config versions in the ImageWorks repository and keep a single one that
would maybe be a *git-submodule* hosted on an AMPAS dedicated repository.
@aforsythe <https://github.com/aforsythe>, @scoopxyz
<https://github.com/scoopxyz>, any thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#13>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAHaAUAyrA2mMHEIw3yySpiYGECQExpyks5tVkz7gaJpZM4SJLYF>
.
|
Yeah sorry, I was referring to repo size.
Yes absolutely! |
Yes, of course, but there are about 4-5 people doing development work on the repo, the majority use case is just downloading the files to use. I'll need to read a bit more about LFS to see how it integrates into things.
Yes, there was a sidebar conversation that prompted this ticket. Just needs to be done. |
As highlighted in this comment, it should be possible to trim some fat by reducing indentation of the spi1d LUTs, some have 8 or 9 characters which is simply overkill. Cheers, Thomas |
I would be keen on discussing about potentially removing the various ACES config versions in the ImageWorks repository and keep a single one that would maybe be a git-submodule hosted on an AMPAS dedicated repository.
@aforsythe, @scoopxyz, @lgritz (and anybody interested) any thoughts?
The text was updated successfully, but these errors were encountered: