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

Provide setup instructions #66

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

Conversation

AiyionPrime
Copy link

As we wasted some time on figuring out this packages relation with pythons isolation features and otherwise common pip install -e . pattern.

@TimSchimansky
Copy link

This finally helped!

@0xMihir
Copy link

0xMihir commented Dec 7, 2024

I had to add <cstdint> to rasterizer_impl.h as mentioned here:
https://github.com/graphdeco-inria/diff-gaussian-rasterization/pull/49/files

@AiyionPrime
Copy link
Author

AiyionPrime commented Dec 9, 2024

@0xMihir That would be to have gcc13 compatibility, right?
I think it's fine for it to stay in the seperate PR :)

Eventually somebody from @graphdeco-inria or possibly @Snosixtyboo might review some of these PRs :)

But thanks for linking #49 as a heads-up for other researchers :)

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