Skip to content

Conversation

@maanasee
Copy link
Contributor

@maanasee maanasee commented May 4, 2023

Fixes #117

@netlify
Copy link

netlify bot commented May 4, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit c4883a8
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/64641905d1f4a000081e7f4f
😎 Deploy Preview https://deploy-preview-193--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented May 4, 2023

@jstac
Copy link
Contributor

jstac commented May 12, 2023

@maanasee you've done a great job so far!

Let's add the derivation of the maximum likelihood estimators for the lognormal case.

Also, I suggest that we change the exponential distribution estimate into an exercise.

  1. First we tell people the max likelihood estimator.
  2. Then we ask them to compute total tax revenue.
  3. Then we ask them to check the fit and see if it's good or not (using a plot).

So nothing new needs to be added --- it's just a reorganization.

@maanasee Are you able to take care of this in the next few days? Or would you like help?

@maanasee
Copy link
Contributor Author

Hi @jstac thanks!

I'll be fine working on this, will update the PR by tomorrow!

@jstac
Copy link
Contributor

jstac commented May 16, 2023

Thanks @maanasee , nice work, much appreciated :-)

I'll take it from here.

lectures/mle.md Outdated
\end{aligned}
$$
Taking $\log$ on both sides gives us the _log likelihood function_ which is:
Copy link
Contributor

Choose a reason for hiding this comment

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

@maanasee , just so you know, you need a blank line before and after $$.

Example:

The function is

$$ f(x) = 2x $$

This function is differentiable...

@jstac
Copy link
Contributor

jstac commented May 16, 2023

@maanasee , some of the text is not well formed:

image

I think it's best if you build your lecture using Jupyter Book to make sure it looks OK. @HengchengZhang , would you mind to help @maanasee with this?

@jstac
Copy link
Contributor

jstac commented May 16, 2023

@maanasee , I recommend you use $\ell$ (\ell) instead of $l$ in maths expressions. It's harder to confuse with $1$.

@jstac
Copy link
Contributor

jstac commented May 17, 2023

Thanks @HengchengZhang . I see we are both pushing commits. I hope there was no merge problems.

(I actually meant that I'd like you to help @maanasee understand how to view the lecture she is working on using Jupyter Book (build locally and look at the html file in _build) if she's not sure how to do it. But I appreciate you jumping in to help.)

@HengchengZhang
Copy link
Member

Thanks @jstac, I've resolved the conflicts already and just made minor edits.

And please feel free to pin me @maanasee whenever you meet some building issues.

@jstac
Copy link
Contributor

jstac commented May 17, 2023

Thanks @HengchengZhang , much appreciated.

@jstac jstac merged commit f358d2a into main May 17, 2023
@jstac jstac deleted the mle branch May 17, 2023 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants