pumi-pic icon indicating copy to clipboard operation
pumi-pic copied to clipboard

Minor change to search_mesh_3d function definition

Open zhangchonglin opened this issue 1 year ago • 6 comments

Minor change to search_mesh_3d function definition to be consistent with search_mesh_2d function. search_mesh_2d function is defined as: https://github.com/SCOREC/pumi-pic/blob/a9f7f8ec2630d66e68178c9ffe51610f2de2c15e/src/pumipic_adjacency.hpp#L1010-L1020

zhangchonglin avatar Jan 27 '25 22:01 zhangchonglin

Thank you @zhangchonglin.

Our automated testing is having issues at the moment. Are all tests passing (that you ran with your specific config) with this change?

cwsmith avatar Jan 28 '25 02:01 cwsmith

@cwsmith: all tests are passing except 31, which I think was not passing previously.

The following tests FAILED:
	 31 - moller_trumbore_test (Failed)

zhangchonglin avatar Jan 28 '25 02:01 zhangchonglin

@Fuad-HH I believe you were doing some work with that particular test. Can you comment on its current status?

jacobmerson avatar Jan 29 '25 07:01 jacobmerson

@Fuad-HH I believe you were doing some work with that particular test. Can you comment on its current status?

It passed with my configuration (with both Kokkos CUDA and OpenMP) and, if I remember correctly, it passed the automated testing too. If I could get the terminal output for the test, I would be able to guess what's going wrong there. My first guess will be the pumipic-data repository doesn't have the necessary meshes. Please try checking if the directory contains the latest commits from this repository: https://github.com/SCOREC/pumipic-data

Fuad-HH avatar Jan 29 '25 15:01 Fuad-HH

Updating the https://github.com/SCOREC/pumipic-data, all tests are passing.

zhangchonglin avatar Jan 29 '25 17:01 zhangchonglin

I have built GITRm with the latest software stack, Once I am able to run, I will do the check

dhyan1272 avatar Jan 29 '25 22:01 dhyan1272

@zhangchonglin Would you mind merging master into this branch or rebasing on master?

cwsmith avatar Sep 12 '25 17:09 cwsmith