Bug in createOcTreeFromImage?

[lruthotto]

try if this works for you!

using JOcTree
n     = [16;16;16];
Img   = zeros(UInt8,tuple(n...));
Img[9:11,8:10,6:12] = 1;
tol   = .001;
S     = createOcTreeFromImage(Img,tol)

for me it does not. there are two issues. first Mesh.sparse3 does not exist. second, getindex does not seem to work with UInt's. For a fix see my forked version. Feel free to pull that over if you like it.

[dwfmarchant]

This doesn't work for me either - I don't think it's a hard fix, it just doesn't look like the code has been updated since jOcTree was split from jInv.Mesh or since 0.5.

Is this a piece of code that is useful to anyone? It's not one I have ever used before aside from maybe demonstrating how an OcTree works. Maybe it should be modified to be more flexible in it's inputs?

[eldadHaber]

Its very useful. I'll try to fix it next week E

…

On Tue, Mar 14, 2017 at 9:40 AM, David Marchant ***@***.***> wrote: This doesn't work for me either - I don't think it's a hard fix, it just doesn't look like the code has been updated since jOcTree was split from jInv.Mesh or since 0.5. Is this a piece of code that is useful to anyone? It's not one I have ever used before aside from maybe demonstrating how an OcTree works. Maybe it should be modified to be more flexible in it's inputs? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AF7sexXc-Imgb-_xqHqnVEfLqeeqjrm3ks5rlsL1gaJpZM4KXWxe> .

[dwfmarchant]

No worries - I fixed it this morning. I'll submit a PR with the changes later today.