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

Frequentist or Bayesian estimator for Model Fitting? #20

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

Conversation

b-remy
Copy link
Member

@b-remy b-remy commented Dec 9, 2020

Hi, during Monday 7th's session, we struggled to understand the authors comparison of Frequentist versus Bayesian estimators for their model fitting algorithm (Bayesian Galaxy Shape Measurement for Weak Lensing Surveys -I., Miller et al). So I reproduced and tried to explain their experiment in a notebook, hopping to make it clearer :)

Maybe this could be a Model Fitting's subsection even though I'm not sure it fits the book format. I am ready to make any necessary changes.

@sfarrens
Copy link
Member

Thanks @b-remy, this notebook is very cool. I think with a bit of work this could easily be the start of the model fitting section of the book. We can discuss the details at the next shear book session.

@sfarrens
Copy link
Member

@b-remy Following today's discussion I recommend that you do the following:

  1. Create a new directory in shearbook called appendix.
  2. Put your notebook in that directory.
  3. Choose an appropriate name for the notebook. I don't have any good ideas yet, but I am sure we can come up with something.
  4. In _toc.yml add a new chapter (i.e. part) after Bias and Calibration called Appendix and add the name of your notebook.
- part: Appendix
  chapters:
    - file: appendix/your-notebook

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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.

3 participants