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

Make manifest max_entries a user-configurable parameter #14

Open
arnold-support opened this issue Feb 23, 2023 · 2 comments · May be fixed by #18
Open

Make manifest max_entries a user-configurable parameter #14

arnold-support opened this issue Feb 23, 2023 · 2 comments · May be fixed by #18

Comments

@arnold-support
Copy link

A couple of customers have hit the 100000 limit and got the "Cryptomatte: %lu entries in manifest, limiting to %lu" warning.

Could we make this a user parameter? We can add a warning to the documentation about the pros and cons of setting this limit higher.

@JenusL
Copy link

JenusL commented Nov 7, 2023

Yes this is a needed feature.

@ThiagoIze
Copy link

This is an easy fix (delete some code). Was there a reason for this limit @jonahfriedman ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants