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
Posting this issue to further document efforts by @shreyasahasram08, @tsunhopang, @ThibeauWouters, and myself to achieve better training results for a new Bu2023Ye grid (see #292). The main difference from Bu2022Ye is that the new grid allows Yewind to take values of 0.2, 0.3, and 0.4, while the Bu2022Ye fixed the parameter at 0.3.
We are performing the following tests:
Ensuring parameter parsing from grid filenames works as intended
Running training only for grid files with Yewind = 0.3, which should replicate Bu2022Ye results
We are also exploring multiple areas of potential improvement, including:
Whether values of Yewind parameter are too widely spaced compared to the finer spacing of other parameters
Whether there are enough SVD coefficients to accurately represent the lightcurves (especially since loss curves look good)
Whether the NN architecture can be changed to provide a better mapping between params and SVD coeffs
The text was updated successfully, but these errors were encountered:
I was able to retrain the model and got good inference results, I believe we can now close the issue, but I leave the final decision to the others in this thread (@bfhealy , @shreyasahasram08 , @tsunhopang )
I agree with Thibeau's suggestion. I also retrained the model and made the benchmark plot below, which suggests generally good performance with the high reduced chi2 values coming from a few outliers near the edge of the grid.
Posting this issue to further document efforts by @shreyasahasram08, @tsunhopang, @ThibeauWouters, and myself to achieve better training results for a new
Bu2023Ye
grid (see #292). The main difference fromBu2022Ye
is that the new grid allowsYewind
to take values of 0.2, 0.3, and 0.4, while theBu2022Ye
fixed the parameter at 0.3.We are performing the following tests:
Yewind = 0.3
, which should replicateBu2022Ye
resultsWe are also exploring multiple areas of potential improvement, including:
Yewind
parameter are too widely spaced compared to the finer spacing of other parametersThe text was updated successfully, but these errors were encountered: