-
Notifications
You must be signed in to change notification settings - Fork 182
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
NEWTON_DOUBLE_PRECISION=on broken since f8e1a3c64b70dff7159416e3a8abb7874f547783 #287
Comments
Ah I will fix it, but I can't commit at the moment since I am moving that
functionality to it own file, since it has become more complex.
The is part of the change to the scene manager for gpu.
Basically the build has to be what they call embarrass parallel,
essentially lock and atomic free, and linear scaling.
At first I thought that the gpu will not need it but in practice there are
object that are just to hard to do in gpu, stuff like find joint, raycast,
convex case, and so on, you can offload to gpu because the gou sence is few
frame behind.
So this imply that the cpu need to keep a spatial scene fir the client app
doing queries.
So I start from there. But is has been quite challenging to make and
incremental bvh tree, that is embarrassing parallel.
Hourly I think I have now.
Any way, I will check the double build.
…On Sun, Jul 10, 2022, 11:59 AM Logan Perkins ***@***.***> wrote:
Starting with that commit, building with double precision results in
/tmp/newton-dynamics/newton-4.00/sdk/dCollision/ndScene.cpp:2816:13: error: no matching function for call to 'ndMax'
max_x = ndMax(intPosit.m_ix, max_x);
(and the same on the next 2 lines)
And as of commit a5f11c2
<a5f11c2>
it also has the following error
/tmp/newton-dynamics/newton-4.00/sdk/dCollision/ndScene.cpp:96:5: warning: array index 3 is past the end of the array (which contains 3 elements) [-Warray-bounds]
m_keyCode[3] = 2;
^ ~
/tmp/newton-dynamics/newton-4.00/sdk/dCollision/ndScene.cpp:105:4: note: array 'm_keyCode' declared here
ndUnsigned32 m_keyCode[3];
—
Reply to this email directly, view it on GitHub
<#287>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB6EPJHOJKIVVMJR53KAAVLVTMMQRANCNFSM53FNWTGQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
ok the double precision problem is fixed now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Starting with that commit, building with double precision results in
(and the same on the next 2 lines)
And as of commit a5f11c2
it also has the following error
The text was updated successfully, but these errors were encountered: