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

Presentation fin laura #659

Open
wants to merge 43 commits into
base: devel
Choose a base branch
from
Open

Presentation fin laura #659

wants to merge 43 commits into from

Conversation

lasofivec
Copy link
Collaborator

@lasofivec lasofivec commented Jul 22, 2022

Following changes:

  • update on the cython speedup notebook
  • added small notebook on the computation of the solid angle subtend by a triangle
  • added tex files on the computations of said solid angle (my version and Didier's, different notations)
  • small changes to better test if a string starts or end with a sub string
  • added reference for the computation for the determination if a 3D angle is reflex or not
  • better print for errors on _vignetting_tools.pyx

@lasofivec lasofivec requested a review from Didou09 July 22, 2022 10:14
@lasofivec lasofivec self-assigned this Jul 22, 2022
@lasofivec
Copy link
Collaborator Author

Just waiting for the tests to pass before updating the PR text (sorry for the messiness)

@lasofivec
Copy link
Collaborator Author

Update done, ready when you are @Didou09

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