Skip to content
This repository has been archived by the owner on Sep 21, 2021. It is now read-only.

Update LLN and CLT #34

Merged
merged 2 commits into from
Aug 28, 2018
Merged

Update LLN and CLT #34

merged 2 commits into from
Aug 28, 2018

Conversation

Nosferican
Copy link
Collaborator

No description provided.

@Nosferican Nosferican added this to the Update to 0.7/1.0 syntax milestone Aug 28, 2018
@Nosferican Nosferican self-assigned this Aug 28, 2018
@Nosferican Nosferican requested a review from arnavs August 28, 2018 07:08
@arnavs arnavs mentioned this pull request Aug 28, 2018
53 tasks
@arnavs
Copy link
Member

arnavs commented Aug 28, 2018

Generally looks great. A few things:

  1. What's going on with the title of this plot? (Ignore the dvipng issues, those are probably missing some command-line tool on my local machine).

screen shot 2018-08-28 at 11 25 54 am

  1. For some reason, our version (below) of this plot looks a lot worse. Some issues in the plotting library? Maybe we can take another look.

screen shot 2018-08-28 at 11 31 11 am

  1. The KernelDensity package is throwing some kind of Warning: Could not import LinearAlgebra into Interpolations.jl error. Doesn't seem to break anything...but good to keep an eye on.

@arnavs arnavs merged commit f2242e9 into master Aug 28, 2018
@arnavs arnavs deleted the Update-LLN_CLT branch August 29, 2018 03:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants