Lehman Garrison

Results 150 comments of Lehman Garrison

Thanks. I think that one is probably this bug: https://stackoverflow.com/questions/51290930/why-does-gcc-not-provide-the-avx512f-intrinsic-for-absolute-value-of-floats?rq=1 Looks like we can work around this one as well. That's two workarounds for gcc 5... but gcc 5.5.0 is...

I think there's 3 things to do here: 1) Detect if the CPU and compiler support `-mavx512vl` but `__AVX512VL__` is not present in the preprocessor output for `-march=native`. If so,...

Thanks for the report! `common.mk` should currently print out the following advice: ``` ../common.mk:302: Compiler is Apple clang and does not support OpenMP If you want OpenMP support, please install...

This is pretty weird! Could you share the job submission script that you're using to run this example? I'm wondering if some kind of stderr/stdout buffer could be filling up...

So if you put a Python print statement in the loop, it prints 13 times? Do you get 13 chunks of verbose Corrfunc output too? The Corrfunc output comes from...

If it's convenient for you to run Corrfunc from source, could you try a few wurlitzer-related things? Specifically, in `utils.py`, there's this snippet in `sys_pipes()`: https://github.com/manodeep/Corrfunc/blob/75b8ac98cc5cc76a5d8aa95715937194cec8ea15/Corrfunc/utils.py#L1044-L1055 Could you print the...

Okay, it sounds like the TTY detection is working properly. The interspersed output is likely just due to buffering and is not unusual. Adding flush=True as an argument to print()...

Okay, not seeing a problem yet. The following is a little paranoid, but easy to check. Could you pip install psutil and place the following in the Wurlitzer context: p...

I'm not sure about the number of fds. I guess it is a leak, but this is way under the default max number of fds, which I think is 1024....

Thanks for the report! I can reproduce this bug. It looks like an error in how we populate the interpolation arrays in `set_cosmo_dist()`. Specifically, I think this line: https://github.com/manodeep/Corrfunc/blob/master/utils/set_cosmo_dist.c#L44 should...