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

Adding phase retrieval #130

Open
wants to merge 159 commits into
base: develop
Choose a base branch
from

Conversation

tobias-liaudat
Copy link
Member

@tobias-liaudat tobias-liaudat commented Mar 11, 2024

Summary

Closes #67. Closes #91. Closes #132.

Issue #66 requires the physical layer from #123.

This PR adds the phase retrieval capabilities including a new projection algorithm that takes into account the obscurations.

Added unit tests for the new projection function, tf_decompose_obscured_opd_basis, and the original PI_zernikes function.

Validation Test Reports

Reviewer Checklist

Reviewers should tick the following boxes before approving and merging the PR.

  • The PR targets the develop branch
  • The PR is assigned to the developer
  • The PR has appropriate labels
  • The PR is included in appropriate projects and/or milestones
  • The PR includes a clear description of the proposed changes
  • If the PR addresses an open issue the description includes "closes #"
  • The code and documentation style match the current standards
  • Documentation has been added/updated consistently with the code
  • All CI tests are passing
  • API docs have been built and checked at least once (if relevant)
  • All changed files have been checked and comments provided to the developer
  • All of the reviewer's comments have been satisfactorily addressed by the developer
  • All Validation tests for the reported error are passing
  • I have reviewed the CI log to verify if any files require formatting with Black. If any files are found to be improperly formatted, I have provided comments indicating the need for formatting.

jeipollack and others added 30 commits March 20, 2024 10:09
…param and zk_prior directly to determine num of Zernikes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants