search

jsitaraman / pifus

Parallel Interpolation For Unstructured Sets
Apache License 2.0
4 stars 1 forks source link

Incorrect array + index in `dMeshBlock::search_inverse_map`? #3

Open sayerhs opened 4 years ago

sayerhs commented 4 years ago

@jsitaraman I was looking at the source code and noticed the arguments to cutHole seem off

https://github.com/jsitaraman/pifus/blob/00dac4878d416ddc262225b3fe2c4b05b813691e/src/dMeshBlock2.C#L156-L160

Should the second line be xwbmax[0], xwbmax[1], xwbmax[2] instead?

jsitaraman commented 4 years ago

You are right. Yes it should be like that. Don't why it was working.

J

On Sat, Dec 21, 2019, 4:18 PM Shreyas Ananthan notifications@github.com wrote:

@jsitaraman https://github.com/jsitaraman I was looking at the source code and noticed the arguments to cutHole seem off

https://github.com/jsitaraman/pifus/blob/00dac4878d416ddc262225b3fe2c4b05b813691e/src/dMeshBlock2.C#L156-L160

Should the second line be xwbmax[0], xwbmax[1], xwbmax[2] instead?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jsitaraman/pifus/issues/3?email_source=notifications&email_token=ACH3YMAABTB6WIJVWFLEQB3QZ2W33A5CNFSM4J6JMOX2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ICE54TQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACH3YMEPQXJODIVLUYWRIDLQZ2W33ANCNFSM4J6JMOXQ .