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

fix: specify nvdiffrast version as v0.3.0 #186

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WarmCongee
Copy link

Hello, this is a very good work. Your work has been widely used in the field of Audio-driven Talking Head Generation this year to test an indicator called E-FID. "EMO: Emote Portrait Alive - Generating Expressive Portrait Videos with Audio2Video Diffusion Model under Weak Conditions".
When I used this repository to test this indicator, I found an error that may occur during configuration after Jun 30, 2023.

There are three lines of commands in the repository's README.md:

git clone https://github.com/NVlabs/nvdiffrast
cd nvdiffrast
pip install . 

The README was updated on May 26, 2023, when the repository link pointed to nvdiffrast version v0.3.0. Now that the nvdiffrast repository has been updated since Jun 30, 2023, using other environments originally specified in the repository in combination with new version nvdiffrast will cause errors in compiling nvdiffrast_plugin.
Therefore, the corresponding nvdiffrast version should be strictly specified in the README. The correct configuration command should be:

git clone -b v0.3.0 https://github.com/NVlabs/nvdiffrast
cd nvdiffrast
pip install . 

@WarmCongee WarmCongee closed this Oct 19, 2024
@WarmCongee WarmCongee reopened this Oct 19, 2024
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.

1 participant