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

Update Multiplicative Functionals #90

Merged
merged 1 commit into from
Sep 19, 2018
Merged

Update Multiplicative Functionals #90

merged 1 commit into from
Sep 19, 2018

Conversation

Nosferican
Copy link
Collaborator

Needs tests and plots in separate PR.

@Nosferican
Update Multiplicative Functionals
14bbdc9 
Needs tests and plots in separate PR.
@Nosferican Nosferican added this to the Update to 0.7/1.0 syntax milestone Sep 13, 2018
@Nosferican Nosferican self-assigned this Sep 13, 2018
@arnavs arnavs mentioned this pull request Sep 14, 2018
Closed
53 tasks
arnavs
arnavs approved these changes Sep 19, 2018
View reviewed changes
Copy link
Member

@arnavs arnavs left a comment

Choose a reason for hiding this comment

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

Passes a sanity check of eyeballing plots and checking the final number. Will need to add Random.seed!s eventually when we do tests.

@arnavs arnavs merged commit 43ea69b into master Sep 19, 2018
@arnavs arnavs deleted the Update-Multiplicative-Functionals branch September 20, 2018 23:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@arnavs arnavs arnavs approved these changes

Assignees

@Nosferican Nosferican

Labels
None yet
Projects
None yet
Milestone
Update to 0.7/1.0 syntax
Development

Successfully merging this pull request may close these issues.
2 participants
@Nosferican @arnavs