search

aimalz / qp

Quantile Parametrization for probability distribution functions module
MIT License
3 stars 9 forks source link

Issue/43/survey mode #75

Closed aimalz closed 7 years ago

aimalz commented 7 years ago

I implemented only the most basic functionality for survey mode and want to merge it into master so I can start a new branch for the home stretch toward a paper without getting too far from the master branch. There's a lot of deadweight in the data exploration notebook and ensemble class, but I'd like to keep it for that new branch.

aimalz commented 7 years ago

Actually, nevermind, I can do the work I want in a sloppy branch and deal with one really big merge later on. . let's keep this on the back burner until the end of the week.

aimalz commented 7 years ago

I think now is a good time to get the barebones implementation of survey mode into the master branch. @drphilmarshall this pull request needs your review. Thanks!

aimalz commented 7 years ago

@drphilmarshall Are you ready to review this PR? I'd like to have some version of the qp.Ensemble class in the master branch for the sake of others working on the LSST-DESC PZ DC1 paper. I'd plan to leave #43 and its corresponding branch open after the merge for further, necessary development and cleanup later on. What do you think of this plan? Thanks!

drphilmarshall commented 7 years ago

Bravo! :party_parrot: