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

fixed #231 #291 MacOS compatibility #368

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

piterand
Copy link

@piterand piterand commented Jun 15, 2023

Solving #231 #291 #15 #122 #379
salesforce-lavis 1.0.2 and salesforce-lavis 1.0.0 depend on decord which is not supported by MacOS, see the link.
But EVA has created a fork eva-decord https://pypi.org/project/eva-decord/ that fixed the installation issues on MacOS.

@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @piterand to sign the Salesforce Inc. Contributor License Agreement.

@piterand
Copy link
Author

I already signed the agreement the same day I opened the PR, but the salesforce-cla bot doesn't see it for some reason. Can you check, please?
Screenshot from 2023-06-23 11-34-21

@salesforce-cla
Copy link

Thanks for the contribution! Unfortunately we can't verify the commit author(s): root <“@g.com”>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated.

@piterand
Copy link
Author

@dxli94 Greetings,
could you please take a look at the PR, it fixes macOS compatibility issues.

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

Successfully merging this pull request may close these issues.

None yet

2 participants