search

mmp / pbrt-v3

Source code for pbrt, the renderer described in the third edition of "Physically Based Rendering: From Theory To Implementation", by Matt Pharr, Wenzel Jakob, and Greg Humphreys.
http://pbrt.org
BSD 2-Clause "Simplified" License
4.89k stars 1.19k forks source link

pbrt scenes: unable to clone (read error) #303

Closed pprabhu78 closed 3 years ago

pprabhu78 commented 3 years ago

Hi, Trying to clone the pbrt scenes. I get this error:

git.exe clone --progress -v "git://git.pbrt.org/pbrt-v3-scenes" Cloning into 'pbrt-v3-scenes'... Looking up git.pbrt.org ... done. Connecting to git.pbrt.org (port 9418) ... 146.148.96.151 done. fatal: read error: Invalid argument

I tried other protocols, no success. I tried the snapshot, but I can't pull from that either (I get the same error). I tried with tortoise git as well as command line.

mmp commented 3 years ago

Fixed now--apologies for not doing it sooner. (I had to reboot the server.)

pprabhu78 commented 3 years ago

It works. Thank you so much.

As an aside, I have read, enjoyed and recommended the pbrt book for almost 15 years now.

In my former life I was a researcher. We were working on using gpgpu to do ray tracing, photon mapping, etc. And yoru book was a great reference.

Ofcourse, now you can do that using dxr, vulkan. But it continues to be a standard book on whenever I need to look up canon on ray tracing.

-Poojan

On Thu, Oct 29, 2020 at 3:27 PM Matt Pharr notifications@github.com wrote:

Fixed now--apologies for not doing it sooner. (I had to reboot the server.)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mmp/pbrt-v3/issues/303#issuecomment-718971646, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACOF3ZZ6MRFVVGU7JBVODP3SNG63HANCNFSM4S5OOURQ .

mmp commented 3 years ago

Thank you! That's great to hear.

(FWIW the next version [pbrt-v4][https://github.com/mmp/pbrt-v4) now optionally supports the GPU via OptiX and CUDA.)

pprabhu78 commented 3 years ago

That's good to know (about Optix and CUDA)

On Thu, Oct 29, 2020 at 7:57 PM Matt Pharr notifications@github.com wrote:

Thank you! That's great to hear.

(FWIW the next version [pbrt-v4][https://github.com/mmp/pbrt-v4) now optionally supports the GPU via OptiX and CUDA.)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mmp/pbrt-v3/issues/303#issuecomment-719089703, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACOF3Z5YSF334Y46W5A6XQTSNH6PVANCNFSM4S5OOURQ .