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

Consistent file structure for quad #743

Merged

Conversation

mmahsereci
Copy link
Contributor

@mmahsereci mmahsereci commented Nov 21, 2022

In a Nutshell

No changes in functionality. No changes in docs. Simply making file structure consistent among quad modules.

Detailed Description

Should be reviewed after #742 is merged as it branches off of it.

Related Issues

Closes #...

@mmahsereci mmahsereci marked this pull request as draft November 21, 2022 14:36
@mmahsereci mmahsereci self-assigned this Nov 21, 2022
@mmahsereci mmahsereci added the quad Issues related to quadrature label Nov 21, 2022
@codecov
Copy link

codecov bot commented Nov 21, 2022

Codecov Report

Merging #743 (e81d099) into main (77a2eef) will increase coverage by 0.02%.
The diff coverage is 98.19%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #743      +/-   ##
==========================================
+ Coverage   90.48%   90.51%   +0.02%     
==========================================
  Files         199      202       +3     
  Lines        7517     7538      +21     
  Branches      974      974              
==========================================
+ Hits         6802     6823      +21     
  Misses        481      481              
  Partials      234      234              
Impacted Files Coverage Δ
...rc/probnum/quad/solvers/policies/_random_policy.py 100.00% <ø> (ø)
...um/quad/solvers/belief_updates/_standard_update.py 95.55% <95.55%> (ø)
src/probnum/quad/integration_measures/__init__.py 100.00% <100.00%> (ø)
...num/quad/integration_measures/_gaussian_measure.py 100.00% <100.00%> (ø)
.../quad/integration_measures/_integration_measure.py 100.00% <100.00%> (ø)
...num/quad/integration_measures/_lebesgue_measure.py 100.00% <100.00%> (ø)
...rc/probnum/quad/solvers/belief_updates/__init__.py 100.00% <100.00%> (ø)
...bnum/quad/solvers/belief_updates/_belief_update.py 100.00% <100.00%> (+3.50%) ⬆️
...um/quad/solvers/policies/_van_der_corput_policy.py 100.00% <100.00%> (ø)

@mmahsereci mmahsereci marked this pull request as ready for review November 22, 2022 15:38
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.

👍

Co-authored-by: Jonathan Wenger <jonathan.wenger@uni-tuebingen.de>
@mmahsereci mmahsereci merged commit 92e59ad into probabilistic-numerics:main Nov 23, 2022
@mmahsereci mmahsereci deleted the mm-quad-file-structure branch November 23, 2022 11:47
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.

2 participants