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 PSF content and structure #25

Merged
merged 13 commits into from
Feb 26, 2021
Merged

Conversation

tobias-liaudat
Copy link
Member

@tobias-liaudat tobias-liaudat commented Jan 22, 2021

I'm adding a possible structure for the PSF chapter of the book (Issue #7 )

Also I'm adding possible content to the sections that we can discuss on Friday's meeting.

Note: Still unfinished work.

@sfarrens
Copy link
Member

For diffraction effects perhaps an interactive function to plot an Airy disk (e.g. using Astropy function) based on telescope aperture size and/or wavelength.

@tobias-liaudat
Copy link
Member Author

@tobias-liaudat
Copy link
Member Author

I added a new preliminary version of the sources of the PSF with some examples.

We should give a nicer format for the examples and the code, but the content should be there. I could use some help over there.

If you have comments please don't hesitate to leave them here.

@tobias-liaudat tobias-liaudat linked an issue Feb 18, 2021 that may be closed by this pull request
2 tasks
@sfarrens
Copy link
Member

Nice @tobias-liaudat we can go through this tomorrow!

aboucaud
aboucaud previously approved these changes Feb 18, 2021
Copy link
Collaborator

@aboucaud aboucaud left a comment

Choose a reason for hiding this comment

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

Very nice start, I guess this PR should be merged to set the structure and future PR should be opened to add more content to a given file at a time.

@aboucaud
Copy link
Collaborator

I'll let you discuss that tomorrow of course but since I won't be there, you have my opinion 😉

After seeing these nice a visual plots, one thing I would consider adding (although maybe not in this intro notebook) is the Fourier decomposition of the PSF you showed, since convolutions are usually done in Fourier.

@tobias-liaudat
Copy link
Member Author

@aboucaud yes, it is not finished work. I'll wait for tomorrow to merge.

Maybe I'll add a third column to the plots showing the FFT.

@aguinot
Copy link

aguinot commented Feb 18, 2021

Hey!
I just finish looking at your section and I think it looks very nice! We can discuss tomorrow how technical and how much details we want to put here. You can found some comments below. We can discuss more tomorrow.

  • I would move 1.1.2 to 1.1.1.1, it would make more sense to me.
  • 1.1.1.1: For the ground based PSF, I agree with what you say on the Moffat for long exposures, but I don't think we should only show a Moffat.. I would at least show a Kolmogorov model which is based on the atmospheric perturbations theory.
  • 1.1.1.1: I would show the space based before the ground based.
  • 1.1.2.1: Be careful when you say "mirrors are not perfectly spheric". I think this is "maladroit". Some mirrors are not spheric at all on purpose and this actually correct some aberrations. I think it would be better to say that there is some imperfections in the curvature or during the fabrication. In any case I would not refer to the geometry.
  • 1.1.2.1: regarding the size, I agree with your comment but from what you say it seems that we could just increase the size freely. But the larger the mirror (mono-bloc) the harder it is to conserve its shape.. There is actually Active Optic systems that correct for the distortions due to mirror own weight. Maybe this is too technical for this book..
  • 1.1.2.1: for the obscuration, you mention the "arm" but the secondary (or third, or more..) or even the camera (like CFHT, LSST,..) are in front of the primary mirror and create a large obscuration (>30%). And it will have a strong effect as well.
  • 1.1.2.1: In the filter I change "limit" to "select".
  • 1.1.2.2: maybe we can mention the dome turbulence which is a strong contribution to the PSF..
  • 1.1.2.3: Guiding errors are not specific to space observations but it might be harder to correct for it in space (just because moving the telescope is more complicated).
  • 1.1.2.3: Do you have a ref for the polarisation effect?
  • 1.1.2.4: I don't know if I would mentioned all those effects since some of them are "just" artifact that are a problem for the entire analysis, not only the PSF (CDE, CTI, QE).
  • 1.1.2.4: For the under sampling you can link to the "parametric model" section which solve this problem. (Right ?)
  • 1.1.2.4: For the Brighter Fatter, "Each CCD in a camera is not independent...", I agree, but don't you mean "each pixels" ?

@andrevitorelli
Copy link

Computational Fluid Dynamics simulation of GMT enclosure
https://www.youtube.com/watch?v=HU1i6JzvIzY

@thuiop
Copy link

thuiop commented Feb 26, 2021

Wow, these are a lot of figures !
Some criticism :

  • The wavefront aberration figure features a lens, not a mirror, and large telescopes are made with mirrors (and you talk about mirror imperfections in the text). The second one is good.
  • The HST errors one is good. It seems there is a formatting problem just after though, maybe because of indent ?
  • The scattering light one is superfluous to me ; I do not think it helps much to understand that there are some reflections in the mirror. Just citing the paper if people want more details is sufficient.
  • Obscurations and filters are nice.
  • I do not know whether it is possible to have an animated image in a notebook but an animation of the changing atmospheric PSF would be more explicit than the Kolmogorov screens.
  • The environmental one is great.
  • I cannot judge for the Charge Diffusion error ; but there is a typo in the title.

Overall great job finding all those figures !
Side note : don't forget to hide the plotting function at the beginning in the notebook ; you can put it in shrbk/plot.py at the root of the repo and only import it in the notebook.

@tobias-liaudat
Copy link
Member Author

@thuiop Thanks for the comments!

The wavefront aberration figure features a lens, not a mirror, and large telescopes are made with mirrors (and you talk about mirror imperfections in the text). The second one is good.
Yeah, indeed the illustration is of a lens. I wanted to illustrate what optical path differences were but didn't found an example with mirrors :/ I think I'll keep the illustration but specifying more clearly the point you raised.

The HST errors one is good. It seems there is a formatting problem just after though, maybe because of indent ?
I don't see the formatting when I build the notebook. Can you show me what does it look like?

The scattering light one is superfluous to me ; I do not think it helps much to understand that there are some reflections in the mirror. Just citing the paper if people want more details is sufficient.
Yeah, it's not super illustrative. I'll ask the question in today's meeting and maybe erase it.

Obscurations and filters are nice.
Thanks :)

I do not know whether it is possible to have an animated image in a notebook but an animation of the changing atmospheric PSF would be more explicit than the Kolmogorov screens.
Indeed, if that's a possibility, it would be better! Maybe someone else in the group can answer that question..

The environmental one is great.
I cannot judge for the Charge Diffusion error ; but there is a typo in the title.
Thanks, I'll correct it

@thuiop
Copy link

thuiop commented Feb 26, 2021

It seems the formatting problem was in the notebook and it does not reflect in the book, so nevermind about that.

aguinot
aguinot previously approved these changes Feb 26, 2021
@sfarrens sfarrens merged commit 695d072 into CosmoStat:develop Feb 26, 2021
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.

Add content on PSF modelling methods
6 participants