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 some Resolve features #18

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

r2d3
Copy link

@r2d3 r2d3 commented Aug 14, 2023

add OFX::Host::Param::StrChoiceInstance

add ImageEffect properties :

  • kOfxImageEffectPropCudaStreamSupported
  • kOfxImageEffectPropMetalRenderSupported

add Render properties :

  • kOfxImageEffectPropMetalEnabled
  • kOfxImageEffectPropCudaStream
  • kOfxImageEffectPropMetalCommandQueue

correct a use after free error in kOfxActionUnload

add OFX::Host::Param::StrChoiceInstance
add ImageEffect properties :
- kOfxImageEffectPropCudaStreamSupported
- kOfxImageEffectPropMetalRenderSupported
add Render properties :
- kOfxImageEffectPropMetalEnabled
- kOfxImageEffectPropCudaStream
- kOfxImageEffectPropMetalCommandQueue
correct a use after free error in kOfxActionUnload
@r2d3 r2d3 force-pushed the feature/resolve_strchoice branch from 11715df to a9aebbc Compare August 14, 2023 20:45
Copy link
Member

@devernay devernay left a comment

Choose a reason for hiding this comment

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

LGTM!
Thank you David!
Since you seem to be developing on resolve, could you take a look at why the plugins in openfx-misc don't load/run? I found no way to debug this. Probably some openfx stuff that doesn't please Resolve.
Also, we get regular requests for binaries of denoisesharpen (from openfx-misc) for Resolve, do you think you could contribute this?

# endif
stat = (*_pluginHandle)->mainEntry(kOfxActionUnload, 0, 0, 0);
# ifdef OFX_DEBUG_ACTIONS
std::cout << "OFX: "<<op->pluginIdentifier<<"("<<(void*)op<<")->"<<kOfxActionUnload<<"()->"<<StatStr(stat)<<std::endl;
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for fixing that!

Copy link
Member

Choose a reason for hiding this comment

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

It's it still a draft or is it ready to merge?

Copy link
Author

Choose a reason for hiding this comment

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

It is still a draft as I am not sure of the way StrChoiceInstance::notify is supposed to work.

Copy link
Author

Choose a reason for hiding this comment

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

About openfx-misc plugins, what is the issue ? They do not load on macOS and Windows and Linux or a specific OS and specific versions of Resolve ?

@devernay
Copy link
Member

ready?

@r2d3
Copy link
Author

r2d3 commented Nov 11, 2023

ready?

Not yet. I am putting it into action and StrChoice is still incomplete.

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

Successfully merging this pull request may close these issues.

2 participants