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

WIP: use the ode solver for growth factor and growth rate. #344

Merged
merged 5 commits into from Jun 5, 2017

Conversation

Projects
None yet
1 participant
@rainwoodman
Member

rainwoodman commented Jun 3, 2017

this shall close #343

@rainwoodman rainwoodman force-pushed the rainwoodman:perturb branch from 0e30299 to ff44760 Jun 4, 2017

@rainwoodman rainwoodman force-pushed the rainwoodman:perturb branch from 315465b to 931dd02 Jun 5, 2017

do not compute D inside __call__
because growth_function is now a lot more expensive to compute than
before (due to the ODE integrateion).

We can avoid using the fitted version by computing D in advance.

@rainwoodman rainwoodman requested review from nickhand and removed request for nickhand Jun 5, 2017

@rainwoodman rainwoodman merged commit fc1d68f into bccp:master Jun 5, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.2%) to 86.344%
Details

@rainwoodman rainwoodman deleted the rainwoodman:perturb branch Jun 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment