-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Heatmaps Extended #18
Comments
I don't understand the label on the vertical axis. On Fri, Oct 28, 2016 at 11:51 AM, Alexandra La Fleur <
|
I had to shorten the names to avoid an error with the plotting library. (Too long names cannot be plotted) The values 2,4,8 (or 0.125, 1, 8) correspond to the variable values for that heatmap cell. |
Oh, I see - there is some displacement of the labels, but I guess the top row of heatmaps is for eta, the second row is scale_window (could be shortened to swin) and so on. |
We planned to increase the heatmap size anyway. If we do that, there should be more room for the labels. |
Change in strategy on this point - rather than have timescaledisp and statconc on the vertical axis, while taumeta is on the horizontal axis, I would like to have a set of heatmaps for each of these model parameters, with the model parameter on the horizontal axis and the algorithmic parameters (eta, scale_win, numtraj) on the vertical axis. The objective is to investigate the question of how to optimize the algorithm parameters, given the model parameters. First we try to understand this by varying one model parameter at a time. Later we can look for joint affects of several model parameters, if necessary. |
…d with issue #21 because running the statconc scripts resulted in this error
OK, this looks reasonable for a single run. Once we finalize the taumeta scripts, then we will also run these for a large number of runs to get the expected error. |
Also, we can omit the performance calculation since it does not depend on this parameter in any way. |
The effect from statconc is considerably less than the other parameters (note the error scale goes only from -3 to -2). So it is certainly of lower priority., tdisp has a greater effect, but it is somewhat confounded with taumeta, due to the way it was implemented. So this effect would need to be looked at more closely to separate the effect of the spread of timescales from the effect of the magnitude of timescales. So it is put at lower priority because of the effort involved, with likely result that the effect is also negligible. |
The heatmaps that investigate parameters other than taumeta should use the same value of taumeta when generating the data and performing the estimation. Otherwise, more data is generated (by a factor of 2 or more) than is needed, and that makes the computation much slower than necessary. |
We need to understand how timescaledisp and statconc influence the evaluation. So, we would like to have two additional heatmaps for these parameters
Issue #4
The text was updated successfully, but these errors were encountered: