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

run on windows, add point drawing #28

Merged
merged 36 commits into from
Sep 1, 2022
Merged

run on windows, add point drawing #28

merged 36 commits into from
Sep 1, 2022

Conversation

3a1b2c3
Copy link
Contributor

@3a1b2c3 3a1b2c3 commented Aug 26, 2022

No description provided.

@tovacinni
Copy link
Collaborator

Thank you for this super important & useful contrib!!

I tested this on both Windows & Linux and everything seems to work as intended 😁

I just left one small comment which would be great if you can quickly address before we merge this!

@3a1b2c3
Copy link
Contributor Author

3a1b2c3 commented Aug 29, 2022

Somehow i dont see a comment now, maybe it was on the file i removed? Please feel free to push to the branch if thats faster

@tovacinni
Copy link
Collaborator

Here's a screenshot of the comment:

image

(might end up pushing to the branch!)

@3a1b2c3
Copy link
Contributor Author

3a1b2c3 commented Aug 29, 2022

Deleted, moved to https://gist.github.com/3a1b2c3/c30ba5b3689680af04583c5bb9a0c083 in case somebody has trouble with environment. I couldn't get it to work wit CUDA 11.6 because of pytorch C bindings I think. Did that work for you?

@tovacinni
Copy link
Collaborator

LGTM, thanks! We can look into the issue with 11.6

@tovacinni tovacinni merged commit df9e4d6 into NVIDIAGameWorks:main Sep 1, 2022
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.

3 participants