You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@KateBrodie@mpalmsten@jstanleyx@bergsmaE - I've been doing a lot of work on cBathy and will send out a summary in a while. But I stumbled on a small item that slows the code a ton. If you don't have the stats toolbox, the code should opt for using the alternate LMFit routine. It knows this by asking for ver('stats'). The bad news is that it asks this for every tile while doing bathyFromKAlpha.m and again in bathyCI.m (but strangely enough, not in csmInvertKAlpha.m. When I did a run time profile I found just these 'ver' calls add about 90 s to a typical cBathy analysis. As an alternate, I suggest adding
if isempty(ver('stats'))
bathy.params.nlinfit = 0;
end
at the beginning of analyzeBathyCollect.m so it is only called once. Then change the previous checks in bathyFromKAlpha.m and bathyCI.m to just check for the params.nlinfit flag.
I would do this change but my code is so different from this version that I will do it as V1.3 when I tidy things up. And I haven't seriously tested this fix.
The text was updated successfully, but these errors were encountered:
@KateBrodie @mpalmsten @jstanleyx @bergsmaE - I've been doing a lot of work on cBathy and will send out a summary in a while. But I stumbled on a small item that slows the code a ton. If you don't have the stats toolbox, the code should opt for using the alternate LMFit routine. It knows this by asking for ver('stats'). The bad news is that it asks this for every tile while doing bathyFromKAlpha.m and again in bathyCI.m (but strangely enough, not in csmInvertKAlpha.m. When I did a run time profile I found just these 'ver' calls add about 90 s to a typical cBathy analysis. As an alternate, I suggest adding
if isempty(ver('stats'))
bathy.params.nlinfit = 0;
end
at the beginning of analyzeBathyCollect.m so it is only called once. Then change the previous checks in bathyFromKAlpha.m and bathyCI.m to just check for the params.nlinfit flag.
I would do this change but my code is so different from this version that I will do it as V1.3 when I tidy things up. And I haven't seriously tested this fix.
The text was updated successfully, but these errors were encountered: