Skip to content
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

Drizzlepac/HAP: test new alignment parameters in SVM regression test run #1942

Open
stscijgbot-hstdp opened this issue Jan 27, 2025 · 1 comment

Comments

@stscijgbot-hstdp
Copy link
Collaborator

Issue HLA-1422 was created on JIRA by Steve Goldman:

I've asked Matt to generate a run of SVM regression tests to see the effects of the new alignment parameters. I (along with instrument scientists) will see how these changes affect the alignment and catalogs.

@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Steve Goldman on JIRA:

I've been testing the impact of the new alignment configuration parameters along with Jennifer Mack. Attached is a csv list of important header keywords in the total SVM products the matt ran, both for the baseline directory (photutils build), and our alignment configuration parameters. 

data location:

alignment configuration paramters: /ifs/archive/dev/processing/hla/home/mburger/singlevisits/results_2025-01-29 

photutils baseline: /ifs/archive/dev/processing/hla/home/mburger/singlevisits/results_2025-02-03```
You can see from the first few lines of the table that we are seeing what we expect. The first row shows 4 nmatches and a SVM_GaiaEDR3 WCS solution. The new alignment configuration parameter change requires a higher nmatch threshold of at least 12 for absolute fitting. We see 12 nmatches and an IMG_GaiaEDR3 WCS solution. The later solution is from pipeline processing, the former, from SVM processing. The second row shows a similar pattern, and so on.

 

I also see slight changes in the RMS_RA and RMS_DEC when the same WCS solution is achieved, and some cases where the same WCS solution is achieved, but with a different number of nmatches. 

This can be explained by the fact that the alignment configuration parameter branch did not include the photutils changes. 

In conclusion, the alignment parameter changes look good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant