Results 262 comments of Cameron Smith

@a-jp The PR is ready for review: https://github.com/a-jp/core/pull/1

All CGNS tests (`ctest -R cgns`) pass with address sanitizer enabled (`-fsanitize=address`, GCC 7.4).

For the record, @KennethEJansen and I discussed the hand-coded dmg approach, and a workflow using `mdlConvert` starting from a Simmetrix/Parasolid model (as @jacobmerson pointed out above). The first step towards...

Thanks @jacobmerson. Do you see any downside to making C++11 required? If not, we should consider/discuss the changes here: https://github.com/SCOREC/core/tree/c++11Required . Issue #298 discusses some ctest failures with the XL...

Given #345 I think this can be closed. If openmpi C++ interface warnings are still an issue we should create a new pr to fix it.

@Thomas-Ulrich Thank you for reporting this and providing all the details. It looks like we are hitting this warning/error described here: https://github.com/open-mpi/ompi/issues/5157 I tried, and failed, to reproduce this on...

@KennethEJansen is using ``` $ gcc --version gcc (GCC) 8.3.0 20190222 (Cray Inc.) ``` and the core branch `addNFath` https://github.com/cwsmith/core/blob/7afeab7305e6ce47b50c8bab37db8e517dd5be13/gmi/gmi_analytic.c

@uphoffc Thank you. I was able to reproduce the error. For the record, docker used GCC 8.3.0.

I merged the apf_sim_config branch into develop, created a build dir, ran `./nightlyConfigure.sh on` cmake script on avatar, and then ran make. The build went through without errors. The configure...

with ``` $ clang --version Apple LLVM version 8.0.0 (clang-800.0.42.1) ``` c39c4d1 gets me through the build