Skip to content

Fix/read noise#55

Open
rjmassey wants to merge 27 commits intomainfrom
fix/readNoise
Open

Fix/read noise#55
rjmassey wants to merge 27 commits intomainfrom
fix/readNoise

Conversation

@rjmassey
Copy link
Collaborator

No description provided.

dlagattuta and others added 27 commits November 2, 2022 14:06
estimating covariance matrices for different regions of chip, now returning both estimated readnoise fraction and full covariance matrices
Adding quick comment, checking to see if git push works automatically now
excluding wrapper.cpp
refining covariance matrix calculation
Refactored/optimized readnoise process
added initial plotting function and cleaned covariance matrix
initial re-factoring / cleaning of the code to make the interface more user-friendly.

Also added a convenience function (set_arctic_parameters) to save Arctic CTI parameters ahead of time, instead of user-inputting them in each analysis function
more cleanup: removed redundant functions, added better comments

Also included a function for running the S+R optimisation using an astronomical data frame instead of pure simulation
Additional refactoring of the code, better specifying front-facing functions from back-end ones.

Also improved documentation
refactoring code, adding example usage cases in documentation
fixed self-referential error
more refactoring, additional use-case comments
cleaned read_noise.py, and added a first unit test for the read_noise class
fixed small syntax errors
changed sensitivity limts
added improved documentation to the S+R generator function _determine_noise_model
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