Jonathon Misiewicz
Jonathon Misiewicz
I vote deprecate, _just in case_ we change our mind on this decision.
The "correct" solution is probably to replace PSIO with hdf5. I'd like to talk about that at PsiCon.
Further discussion of replacing `libpsio` should be moved to a new issue. As for your specific proposal: I'm not familiar with Q5Cost, and a quick Google search gives me a...
MBIS is Georgia Tech's department, so obligatory @loriab ping - I'm not sure who has taken over MBIS now that Jeff has left. Is there a legitimate reason for us...
@loriab, I'm not comfortable merging this in while the `dfocc` cleanup is ongoing. Your thoughts?
Please focus on other shards. I recommend discussing again once #2669, #2670, and the next PR in that series (not currently pushed, will be about merging in non-FNO methods contributed...
FTR, David and I have agreed to pause this PR until after #2682 and #2665 come in.
``` rows_per_bucket = -1622894080 nbuckets = -18256 rows_left = -1312595486 ``` This immediately makes me suspect a type overflow error. Thanks for everything thus far.
Just pointing out that this PR doesn't remove C-side optking code yet. Is that coming?
Please ping me for review once Lori approves. She knows this part of the code better than I do.