Closed charlesrp closed 2 years ago
I missed that this PR was against "master". We need to change it to "siaf-updates" before merging and starting the CR process.
@Witchblade101 just changed the base to siaf-updates
. If you want to re-review because it can cause history gaps.
History looks good so I'll go ahead and merge.
Note: generate_nirspec.py results in two assert errors that are benign - improper field points tested for NRS. The current test routines inappropriately test locations outside the valid range. Fixed test code gives good results.