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.

Sign up for GitHub

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

Jump to bottom

tickets/DM-15756 Fix biasCorr code and default params #8

Merged
merged 3 commits into from
Dec 18, 2018
Merged

tickets/DM-15756 Fix biasCorr code and default params #8

merged 3 commits into from
Dec 18, 2018

Commits on Nov 6, 2018

  1. Change code to not divide by biasCorr when running simulation 

    This operation is, as now documented, clearly not appropriate
when the function is being used to calculate the bias.
    @mfisherlevine
    mfisherlevine committed Nov 6, 2018
    Configuration menu
    Copy the full SHA
    ec27fac View commit details
    Browse the repository at this point in the history

Commits on Dec 18, 2018

  1. Remove the biasCorr simulation code 

    Now that the task code agrees with the sim code,
remove it, its uses, and update docs to reflect
its removal.
    @mfisherlevine
    mfisherlevine committed Dec 18, 2018
    Configuration menu
    Copy the full SHA
    476240b View commit details
    Browse the repository at this point in the history

  2. Remove comment after investigating following DM-15756

    @mfisherlevine
    mfisherlevine committed Dec 18, 2018
    Configuration menu
    Copy the full SHA
    c80c077 View commit details
    Browse the repository at this point in the history