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.

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

add the functionality to evaluate the model using boundry values of p… #355

Merged
merged 17 commits into from
Nov 10, 2023

Conversation

EmadAlamoudi
Copy link
Collaborator

closing #353

@codecov-io
Copy link

codecov-io commented Oct 21, 2020

Codecov Report

Merging #355 into develop will increase coverage by 33.56%.
The diff coverage is 32.50%.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop     #355       +/-   ##
============================================
+ Coverage    49.65%   83.21%   +33.56%     
============================================
  Files           87       88        +1     
  Lines         5017     5057       +40     
============================================
+ Hits          2491     4208     +1717     
+ Misses        2526      849     -1677     
Impacted Files Coverage Δ
pyabc/external/base.py 71.81% <23.33%> (-18.19%) ⬇️
pyabc/external/utils.py 60.00% <60.00%> (ø)
pyabc/sampler/multicore_evaluation_parallel.py 100.00% <0.00%> (+1.63%) ⬆️
pyabc/sampler/base.py 94.82% <0.00%> (+1.72%) ⬆️
pyabc/random_variables.py 65.87% <0.00%> (+2.38%) ⬆️
pyabc/sampler/redis_eps/cli.py 80.71% <0.00%> (+2.85%) ⬆️
pyabc/storage/bytes_storage.py 100.00% <0.00%> (+3.33%) ⬆️
pyabc/inference/smc.py 94.84% <0.00%> (+3.86%) ⬆️
pyabc/distance/base.py 83.33% <0.00%> (+4.16%) ⬆️
... and 50 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10dfe65...158c8dc. Read the comment docs.

@yannikschaelte
Copy link
Member

@EmadAlamoudi Should I review this?

@EmadAlamoudi
Copy link
Collaborator Author

It is done by me. However, I'm not sure how beneficial this to other users, what do you think?

@yannikschaelte
Copy link
Member

if it helps, it never hurts to have some utility functions somewhere

@yannikschaelte
Copy link
Member

@EmadAlamoudi could you resolve the merge conflicts? gonna have a look then.

@yannikschaelte
Copy link
Member

@EmadAlamoudi what is the status here?

Copy link
Member

@yannikschaelte yannikschaelte left a comment

Choose a reason for hiding this comment

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

merge-in needed to resolve conflicts

@codecov-commenter
Copy link

codecov-commenter commented Jun 23, 2022

Codecov Report

Merging #355 (48f3e4c) into develop (6446a77) will increase coverage by 0.25%.
Report is 227 commits behind head on develop.
The diff coverage is 89.70%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff             @@
##           develop     #355      +/-   ##
===========================================
+ Coverage    84.08%   84.34%   +0.25%     
===========================================
  Files           91      141      +50     
  Lines         5260     8535    +3275     
===========================================
+ Hits          4423     7199    +2776     
- Misses         837     1336     +499     
Files Coverage Δ
pyabc/__init__.py 100.00% <100.00%> (ø)
pyabc/acceptor/__init__.py 100.00% <100.00%> (ø)
pyabc/acceptor/pdf_norm.py 96.55% <100.00%> (ø)
pyabc/copasi/__init__.py 100.00% <100.00%> (ø)
pyabc/cv/__init__.py 100.00% <100.00%> (ø)
pyabc/cv/bootstrap.py 100.00% <100.00%> (ø)
pyabc/cv/powerlaw.py 100.00% <100.00%> (ø)
pyabc/distance/__init__.py 100.00% <100.00%> (ø)
pyabc/distance/kernel.py 95.60% <100.00%> (+0.02%) ⬆️
pyabc/epsilon/__init__.py 100.00% <100.00%> (ø)
... and 128 more

@stephanmg
Copy link
Collaborator

@yannikschaelte what's left here?

Use correct comparison operator.
@stephanmg stephanmg removed the request for review from yannikschaelte November 8, 2023 15:15
Copy link
Collaborator

@arrjon arrjon left a comment

Choose a reason for hiding this comment

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

LGTM

@arrjon arrjon removed the request for review from yannikschaelte November 10, 2023 10:12
@stephanmg stephanmg merged commit d689b20 into develop Nov 10, 2023
11 checks passed
stephanmg added a commit that referenced this pull request Nov 10, 2023
* Fix assets for pyabc visserver.

* Update R.

* Fix for logo path in abc-server-dash

* Preparing release notes for next patch release.

* Updating about.rst.

* Updating changelog.

* Create plotly versions of selected matplotlib visualizations (#610)

* Create ploty versions of selected matplotlib visualizations

* update r install instructions

* fix pandas float deprecation warnings

* fixarray ndim > 0 float extraction deprecationwarning

---------

Co-authored-by: Stephan Grein <[email protected]>

* add the functionality to evaluate the model using boundry values of p… (#355)

* add the functionality to evaluate the model using boundry values of parameter

* fix flake8 issues

* fix additional flake8 issues

* add test case

* flake8 fixes

* flake8 fixes

* flake8 fixes

* flake8 fixes

* flake8 fixes

* black fixes

* Update test_external.py

Use correct comparison operator.

---------

Co-authored-by: Yannik Schälte <[email protected]>
Co-authored-by: Stephan Grein <[email protected]>
Co-authored-by: Stephan Grein <[email protected]>

---------

Co-authored-by: Yannik Schälte <[email protected]>
Co-authored-by: Emad Alamoudi <[email protected]>
stephanmg added a commit that referenced this pull request Nov 10, 2023
* Fix assets for pyabc visserver.

* Update R.

* Fix for logo path in abc-server-dash

* Preparing release notes for next patch release.

* Updating about.rst.

* Updating changelog.

* Create plotly versions of selected matplotlib visualizations (#610)

* Create ploty versions of selected matplotlib visualizations

* update r install instructions

* fix pandas float deprecation warnings

* fixarray ndim > 0 float extraction deprecationwarning

---------

Co-authored-by: Stephan Grein <[email protected]>

* add the functionality to evaluate the model using boundry values of p… (#355)

* add the functionality to evaluate the model using boundry values of parameter

* fix flake8 issues

* fix additional flake8 issues

* add test case

* flake8 fixes

* flake8 fixes

* flake8 fixes

* flake8 fixes

* flake8 fixes

* black fixes

* Update test_external.py

Use correct comparison operator.

---------

Co-authored-by: Yannik Schälte <[email protected]>
Co-authored-by: Stephan Grein <[email protected]>
Co-authored-by: Stephan Grein <[email protected]>

---------

Co-authored-by: Yannik Schälte <[email protected]>
Co-authored-by: Emad Alamoudi <[email protected]>
stephanmg added a commit that referenced this pull request Nov 15, 2023
* Fix assets for pyabc visserver.

* Update R.

* Fix for logo path in abc-server-dash

* Preparing release notes for next patch release.

* Updating about.rst.

* Updating changelog.

* Create plotly versions of selected matplotlib visualizations (#610)

* Create ploty versions of selected matplotlib visualizations

* update r install instructions

* fix pandas float deprecation warnings

* fixarray ndim > 0 float extraction deprecationwarning

---------

Co-authored-by: Stephan Grein <[email protected]>

* add the functionality to evaluate the model using boundry values of p… (#355)

* add the functionality to evaluate the model using boundry values of parameter

* fix flake8 issues

* fix additional flake8 issues

* add test case

* flake8 fixes

* flake8 fixes

* flake8 fixes

* flake8 fixes

* flake8 fixes

* black fixes

* Update test_external.py

Use correct comparison operator.

---------

Co-authored-by: Yannik Schälte <[email protected]>
Co-authored-by: Stephan Grein <[email protected]>
Co-authored-by: Stephan Grein <[email protected]>

* Fix extra dependencies.

---------

Co-authored-by: Yannik Schälte <[email protected]>
Co-authored-by: Emad Alamoudi <[email protected]>
Co-authored-by: Jonas Arruda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants