[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. |