Prior Example, closes # 434 #438
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## develop #438 +/- ##
===========================================
- Coverage 91.20% 90.94% -0.26%
===========================================
Files 69 69
Lines 3955 3955
===========================================
- Hits 3607 3597 -10
- Misses 348 358 +10
Continue to review full report at Codecov.
|
yannikschaelte
approved these changes
Aug 6, 2020
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
jvanhoefer
added a commit
that referenced
this pull request
Sep 7, 2020
* Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info>
yannikschaelte
added a commit
that referenced
this pull request
Sep 7, 2020
* Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info>
yannikschaelte
added a commit
that referenced
this pull request
Oct 5, 2020
* V 021 (#455) * Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * fix flake8 Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info>
jvanhoefer
added a commit
that referenced
this pull request
Oct 5, 2020
* Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * V 021 (#455) (#456) * Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Feature cmaes (#457) * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * integration of cma-es optimizer * updated version * updated version * Update setup.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * fix flake8 Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com> * Plot convergence summary and update outdated notebook (#446) * update severely outdated example notebook * add convergence visualization * fix nans * fix flake * update docstring * use problem.get_reduced_vector * fix typehints + docstring * Update pypesto/visualize/optimizer_convergence.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * add to notebook, fix scale, add grad to vis test Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix type checks visualization reference points (#460) * fix for new amici release (#469) * fix for new amici release * update amici requirement * allow longer travis wait time Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com> * fix arguments dlib (#466) (closes #465) * fix arguments ipopt * update docu minimize fct. * fix test_sbml_conversion.py Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> * numpy based subindexing in amici_util (#462) * Update amici_util.py * add return doc * fixups * fix test * fix for non-unique par_opt_slice * reduce function complexity * add check for installation ipopt (#470) closes #468 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> * check optional requirements amici/petab in PEtab import (#477) * add maxiter to Dlibs default options fixes #474 (#476) * add maxiter to Dlibs default options * Update pypesto/optimize/optimizer.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix y_limits in waterfall plot for list of result objects (#475) * prepare v022 (#479) * prepare v022 * fix review suggestions * Fix merge conflicts (#481) * V 021 (#455) * Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * fix flake8 Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: PhilippStaedter <42141163+PhilippStaedter@users.noreply.github.com> Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com>
jvanhoefer
added a commit
that referenced
this pull request
Jan 18, 2021
* release v0.2.2 (#480) * Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * V 021 (#455) (#456) * Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Feature cmaes (#457) * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * updated python script with not yet finished integration of cma-es optimizer * integration of cma-es optimizer * updated version * updated version * Update setup.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * updated version * fix flake8 Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com> * Plot convergence summary and update outdated notebook (#446) * update severely outdated example notebook * add convergence visualization * fix nans * fix flake * update docstring * use problem.get_reduced_vector * fix typehints + docstring * Update pypesto/visualize/optimizer_convergence.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * add to notebook, fix scale, add grad to vis test Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix type checks visualization reference points (#460) * fix for new amici release (#469) * fix for new amici release * update amici requirement * allow longer travis wait time Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com> * fix arguments dlib (#466) (closes #465) * fix arguments ipopt * update docu minimize fct. * fix test_sbml_conversion.py Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> * numpy based subindexing in amici_util (#462) * Update amici_util.py * add return doc * fixups * fix test * fix for non-unique par_opt_slice * reduce function complexity * add check for installation ipopt (#470) closes #468 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> * check optional requirements amici/petab in PEtab import (#477) * add maxiter to Dlibs default options fixes #474 (#476) * add maxiter to Dlibs default options * Update pypesto/optimize/optimizer.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix y_limits in waterfall plot for list of result objects (#475) * prepare v022 (#479) * prepare v022 * fix review suggestions * Fix merge conflicts (#481) * V 021 (#455) * Parameters plot, #418 (#425) * parameters plot, plotting a subset of parameters (#418) * "Variable used before assignment" fixed for parameters plot (#364) * Feature autocorrelation - close #251 (#426) * autocorrelation calculation by sokal * docs * notebook using ess * added ess and ac in result * created ess calculation * rename variable * docs * fix * remove print from test * parameter index in 1d marginal plot * added type * added type * docu * flake8 * Update pypesto/sample/diagnostics.py Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * docs * docs * grammar thingy * handle in get_data_to_plot in visu Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * fix display for least squares solver (#430) fix #429 * basic sanity checking for parameter fixing (#420) * basic sanity checking for parameter fixing * fixup type checks * fixup * fix typehints * fixup & refactor * update doc * fixup * Fix amici error output, fixes #427 (#428) * initial fix * add filter function * unify error and base initialization of return values * Profile options (#419) * change profile indexing to parameter indices instead of boolean indexing * adapt also approximation and test to new profile api Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Prior Example, closes # 434 (#438) * Add notebook prior definition * fix sphinx * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/example/prior_definition.ipynb Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Improve notebook Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Fix offset_y and scale_y in waterfal plot (#441) * Correct offset_y for waterfall plots of result lists * enforce y_limits which show whole data when plotting result lists, if no explicit user input was given * fix nans and infs, deal with possibly different number of optiimzation runs in different result objects * fix 2 typos (#444) Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * Update priors.py (#448) * Fix optimization with priors (#452) * Update priors.py * Update priors.py * Fix PEtab import (Priors) (#454), closes #447 * bugfix + compliance to current PEtab version in prior import * resolve review + flake8 Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * prepare version 0.2.1 * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> * Update doc/releasenotes.rst Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> * fix flake8 Co-authored-by: Jakob Vanhoefer <38346459+jvanhoefer@users.noreply.github.com> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: PhilippStaedter <42141163+PhilippStaedter@users.noreply.github.com> Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com> * fix dublicate * rerun github actions Co-authored-by: Polina Lakrisenko <p.lakrisenko@gmail.com> Co-authored-by: Elba Raimúndez Alvarez <elba.raimundez@helmholtz-muenchen.de> Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com> Co-authored-by: Fabian Fröhlich <fabian@schaluck.com> Co-authored-by: Paul Stapor <paul.stapor@helmholtz-muenchen.de> Co-authored-by: Thomas S. Ligon <tom@thomassligon.info> Co-authored-by: PhilippStaedter <42141163+PhilippStaedter@users.noreply.github.com> Co-authored-by: yannikschaelte <yannik.schaelte@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.