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

OFX::Host::ImageEffect::Base::supportsTiles returns an incorrect value when kOfxImageEffectPropSupportsTiles is set on the plugin rather than the instance #177

Open
adambowen-foundry opened this issue Dec 4, 2024 · 0 comments

Comments

@adambowen-foundry
Copy link

Problem

Plugins can declare support (or not) for tiles by setting kOfxImageEffectPropSupportsTiles on either the plugin descriptor or the plugin instance (per the docs).

If a plugin explicitly sets the property to false on the plugin descriptor (in kOfxActionDescribe ) but sets no value on the instance (in kOfxImageEffectActionDescribeInContext) then supportsTiles returns true even though the plugin has declared that it does not support tiling.

Expected behaviour

If a plugin sets kOfxImageEffectPropSupportsTiles to false on the plugin descriptor, then OFX::Host::ImageEffect::Base::supportsTiles should return false.

Actual behaviour

If a plugin sets kOfxImageEffectPropSupportsTiles to false on the plugin descriptor, then OFX::Host::ImageEffect::Base::supportsTiles returns true.

Notes

I believe this is because a default is declared in the instance property set, and this default overrides the value that would otherwise be inherited from the plugin descriptor. I think it would be more correct for kOfxImageEffectPropSupportsTiles to be treated the same way as kOfxImageEffectInstancePropSequentialRender, and set the initial value to the one from the plugin descriptor.

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

1 participant