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

Get processCcd.py working #9

Merged
merged 4 commits into from
Mar 26, 2020
Merged

Conversation

danjampro
Copy link

A few changes to enable processCcd.py to run without error.

  • We seem to be getting a large amount of cosmic ray detections. I'm not sure whether they are real or whether they correspond to hot pixels, or whether there is something wrong in the config at this point.

  • The PSF modelling seems a bit temperamental. Sometimes it throws an error that no sources are input into the PSF measuring code. Not sure why this is at the moment.

@danjampro danjampro requested a review from fergusL March 26, 2020 03:48
This was referenced Mar 26, 2020
Copy link

@fergusL fergusL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and have made issues for the PSF and cosmic ray tracing

config.repair.doCosmicRay=True
# We seem to be getting too many CR detections
# For now we can increase the max number of allowable CRs
config.repair.cosmicray.nCrPixelMax = 1000000
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

created an issue (#10) to track discussion for this issue


# Sometimes there are no initial source detections, causing errors
# Lowering this number gives more sources for PSF fitting (default 10)
config.detection.includeThresholdMultiplier = 5.0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Created issue (#11) to track discussion for this issue

@fergusL fergusL merged commit 4173691 into AstroHuntsman:develop Mar 26, 2020
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

Successfully merging this pull request may close these issues.

2 participants