Skip to content

Conversation

@edwinnglabs
Copy link
Collaborator

@edwinnglabs edwinnglabs commented Apr 10, 2021

Description

In prediction plot, we will always have a vertical line even if it is in-sample plot. Also make some adjustment of the condition of doing line plot for both in-sample and oo-sample

Fixes #427

  • Bug fix

@edwinnglabs edwinnglabs changed the title Feat plot fix Prediction Plot Bug Fix and Enhance Apr 10, 2021
@edwinnglabs edwinnglabs self-assigned this Apr 10, 2021
@edwinnglabs edwinnglabs added bug Something isn't working review needed need someone to review labels Apr 10, 2021
Copy link
Contributor

@wangzhishi wangzhishi left a comment

Choose a reason for hiding this comment

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

LGTM

@edwinnglabs edwinnglabs merged commit 10423f8 into dev Apr 10, 2021
@edwinnglabs edwinnglabs deleted the feat-plot-fix branch April 10, 2021 20:10
edwinnglabs pushed a commit that referenced this pull request Apr 10, 2021
* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py
edwinnglabs pushed a commit that referenced this pull request Apr 10, 2021
* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py
edwinnglabs pushed a commit that referenced this pull request Apr 10, 2021
* resolve conflicts

* resolve conflicts

* rebase commit

* Prediction Plot Bug Fix and Enhance (#428)

* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py

* rebase commit
edwinnglabs pushed a commit that referenced this pull request Jun 3, 2021
* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py
edwinnglabs pushed a commit that referenced this pull request Jun 3, 2021
* resolve conflicts

* resolve conflicts

* rebase commit

* Prediction Plot Bug Fix and Enhance (#428)

* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py

* rebase commit
edwinnglabs pushed a commit that referenced this pull request Jun 3, 2021
* Prediction Plot Bug Fix and Enhance (#428)

* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py

* Predicion refactor (#430)

* resolve conflicts

* resolve conflicts

* rebase commit

* Prediction Plot Bug Fix and Enhance (#428)

* Update plot.py

* fix bug and refine prediction plot

* Update quick_start.ipynb

* Update plot.py

* rebase commit

* Refine KTRLite plotting and tutorial (#431)

* rebase from dev

* temp

* Update build_your_own_model.ipynb

* refine tutorials and ktrlite knots plotting

* Update ktrlite.ipynb

* palette and docstring

* more verbiage

Co-authored-by: Zhishi Wang <zhishiw@uber.com>

* Update build_your_own_model.ipynb

* Update build_your_own_model.ipynb

* Feat arviz (#433)

* arviz

* get_posterior_samples

* arviz plotting

* arviz demo

* rename

* fix kwargs

* refine plotting

* more robust way to extract signatures of orbit models (#438)

* more robust way to extract signatures of orbit models

* unit test for grid search

* replace regressor beta prior tuning with other params

* ktr lite house keeping (#440)

* ktr lite house keeping

- revert level knots plot
- refine tutorial
- refine some plotting code

* more house keeping

- [x] enhance unit test
- [x] relable pool
- [x] initializer of ktrlite

* notebook update

* Update test_ktrlite.py

* Update test_ktrlite.py

* add comment for stan_extract cleaning

Co-authored-by: Zhishi Wang <zhishiw@uber.com>

* ELBO Loss Extraction (#443)

* refine estimator to include training metrics for NUTS and SVI

* Update pyro_basic.ipynb

* add comment for stan_extract cleaning

Co-authored-by: Zhishi Wang <zhishiw@uber.com>

Co-authored-by: Zhishi Wang <wangzhishi@users.noreply.github.com>
Co-authored-by: Zhishi Wang <zhishiw@uber.com>

* use newly written arviz wrappers for diagnostic tutorial (#446)

* resolve conflicts

* rebase

* rebase with dev

* Update template.py

Co-authored-by: Zhishi Wang <zhishiw@uber.com>
Co-authored-by: Zhishi Wang <wangzhishi@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working review needed need someone to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

In sample prediction get a vertical line

3 participants