#1195: Modify bisection algorithm in EOS_Omni's nuc_eos ------------------------------------+--------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: review Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+---------------------------------------
Comment (by knarf):
Wouldn't it work best if we would bisect not on any real-type variable, but on the index space of the table (of course always looking at that variable in question at the index you are currently at)? That would make it independent of how that variable is distributed within the table. I only had a glance at the patch but it seems to include a lot of calls to log that haven't been there before, and which might mean a performance hit (which might or might not be counter-acted by a different walk through the table).