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 Bq test problems #838

Merged

Conversation

katharina-ott
Copy link
Contributor

In a Nutshell

Add test problems for BQ from Genz-family and various others.

Detailed Description

  • includes option to integrate against uniform or Gaussian measure

Related Issues

Closes #585

fxbriol and others added 30 commits December 2, 2021 10:52
Co-authored-by: Marvin Pförtner <[email protected]>
Co-authored-by: Marvin Pförtner <[email protected]>
Co-authored-by: Marvin Pförtner <[email protected]>
… a version of the Genz continuous function transformed to R^d
…s#585)

* rename var -> std in uniform_to_gaussian_quadprob
* use pytest-cases
@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #838 (4e6fff0) into main (b9b7f3c) will increase coverage by 0.29%.
The diff coverage is 99.35%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #838      +/-   ##
==========================================
+ Coverage   91.17%   91.47%   +0.29%     
==========================================
  Files         218      220       +2     
  Lines        8241     8552     +311     
  Branches     1068     1145      +77     
==========================================
+ Hits         7514     7823     +309     
- Misses        499      500       +1     
- Partials      228      229       +1     
Files Coverage Δ
src/probnum/problems/zoo/quad/__init__.py 100.00% <100.00%> (ø)
...probnum/problems/zoo/quad/_quadproblems_uniform.py 100.00% <100.00%> (ø)
...robnum/problems/zoo/quad/_quadproblems_gaussian.py 96.61% <96.61%> (ø)

Copy link
Contributor

@JonathanWenger JonathanWenger left a comment

Choose a reason for hiding this comment

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

Overall looks very good! It'll be great to have these test problems in the library. I just have a few minor comments and a clarification question about the timing of the MC tests.

docs/source/api/problems/zoo.quad.rst Outdated Show resolved Hide resolved
src/probnum/problems/zoo/quad/_quadproblems_gaussian.py Outdated Show resolved Hide resolved
@JonathanWenger JonathanWenger added the quad Issues related to quadrature label Oct 9, 2023
Copy link
Contributor

@JonathanWenger JonathanWenger left a comment

Choose a reason for hiding this comment

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

Thanks! 🎉

@JonathanWenger JonathanWenger merged commit 60c2584 into probabilistic-numerics:main Oct 25, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quad Issues related to quadrature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants