jaycedowell
jaycedowell
So much for that idea. It's really hard to test this when it now only shows up on CI for me. Also, maybe we should do something about all those...
What if you throw in a couple of copies? Say: ``` data = data[:final.shape[0],:2,:] ``` becomes ``` data = data[:final.shape[0],:2,:].copy() ``` ?
I just got a failure for this more recent test suite on `test_read_drx`. Maybe it's not fixed.
I'm torn about this. Addressing warnings is generally good but it feels kind of wrong to take all of the "class" out of the C++ code. Plus, I don't think...
Hey, on my laptop `configure` detects and uses clang! Seriously, though, I wonder if that original C++17/clang issue might have been related to the problem I ran into at https://github.com/ledatelescope/bifrost/pull/173/commits/d748a9a971cc9ea011e346e58ecaf0ad1c3d8705.
I'm going to try it...
I should probably verify that my fix doesn't just disable C++17 filesystem support universally.
Nope, the fix universally disables support. Back to the drawing board...
@telegraphic is this still a thing? It looks like the crash has been fixed but nothing was done about the angle representation.