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

create rays on device when rendering #993

Merged
merged 2 commits into from
Nov 22, 2022

Conversation

zmurez
Copy link
Contributor

@zmurez zmurez commented Nov 21, 2022

When using traj=="filename" camera_ray_bundle is generated on the cpu and then moved to the gpu which can be slow.

Copy link
Contributor

@tancik tancik left a comment

Choose a reason for hiding this comment

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

Nice! I get ~6% speedup

@zmurez
Copy link
Contributor Author

zmurez commented Nov 21, 2022

I think it depends on utilization/hardware... I got no speed up when running render on a server by itself. But when running 8 copies in parallel on the server (1 per gpu), the cpu became the bottleneck and I got 10x speedup after.

@tancik
Copy link
Contributor

tancik commented Nov 21, 2022

Feel free to merge

@zmurez
Copy link
Contributor Author

zmurez commented Nov 22, 2022

Feel free to merge

I don't have merge access.

@tancik tancik merged commit 0264aac into nerfstudio-project:main Nov 22, 2022
@tancik
Copy link
Contributor

tancik commented Nov 22, 2022

I don't have merge access.

I merged for now. Sent you a new invite.

tancik pushed a commit to dozeri83/nerfstudio that referenced this pull request Jan 20, 2023
chris838 pushed a commit to chris838/nerfstudio that referenced this pull request Apr 22, 2023
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