[PIPE2D-1053] Make a full fluxmodeldata package Created: 09/Jun/22 Updated: 24/Jun/22 Resolved: 24/Jun/22 |
|
Status: | Done |
Project: | DRP 2-D Pipeline |
Component/s: | None |
Affects Version/s: | None |
Fix Version/s: | None |
Type: | Task | Priority: | Normal |
Reporter: | sogo.mineo | Assignee: | sogo.mineo |
Resolution: | Done | Votes: | 0 |
Labels: | flux-calibration, model-templates | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: |
![]() ![]() |
Epic Link: | flux calibration |
Reviewers: | price |
Description |
In 1. We have to extend the format of broadband/photometries.fits so that its user can distinguish the spectra in the 6k package from the other interpolated spectra. 2. We have to make a 60k package. We may include in it a file that indicates the hyperparameters used in interpolating the spectra. 3. We have to modify makeFluxModelInterpolator.py so that it will be aware of the new format of broadband/photometries.fits, and it will make interpolator.pickle only from the 6k spectra. I think we should not include interpolator.pickle in the package but each user should run makeFluxModelInterpolator.py to make it, because the pickle depends on scipy's version. I will make the first version of 60k package with the hyperparameters already known to work reasonably. I think I can find better hyperparameters by estimating the goodness of hyperparameters with cross validation, but that will be another issue. |
Comments |
Comment by sogo.mineo [ 22/Jun/22 ] |
Full package is available here: https://hscdata.mtk.nao.ac.jp/hsc_bin_dist/pfs/fluxmodeldata-ambre-20220518-full.tar.xz Judging merely from these two animations, I guess the interpolation was successful. If you simply replace the small package with this one, drp_stella will probably suffer from short of memory. I have yet to revise drp_stella before closing this issue. |
Comment by sogo.mineo [ 23/Jun/22 ] |
Could you review this PR (drp_stella)? |
Comment by sogo.mineo [ 24/Jun/22 ] |
Merged. Thanks for the review. |