Skip to content

Improved unit testing #68

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.

Sign up for GitHub

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

Merged
merged 13 commits into from
Sep 8, 2024
Merged

Improved unit testing #68

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
fa6b44d
Move Line and Ray to new test section
mikeingold Sep 7, 2024
3c56288
Reorganize structure
mikeingold Sep 7, 2024
cd0fe87
Partition Aqua.jl tests into their own section
mikeingold Sep 7, 2024
7692706
Bugfix
mikeingold Sep 7, 2024
71c412f
Update Plane tests
mikeingold Sep 7, 2024
905e67b
Bugfix
mikeingold Sep 7, 2024
5534b7b
Improved Cone tests
mikeingold Sep 7, 2024
445a510
Bugfix
mikeingold Sep 7, 2024
248b32a
Improved ConeSurface tests and bugfix
mikeingold Sep 8, 2024
f45d7bb
Bugfix - use of wrong default algorithms for surface and volume
mikeingold Sep 8, 2024
246c5fb
Re-implement FrustumSurface tests
mikeingold Sep 8, 2024
6d549b9
Rename new section and clean up some formatting
mikeingold Sep 8, 2024
ad2fbe9
Update status for Frustum, add links to relavent Issues
mikeingold Sep 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions docs/src/supportmatrix.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,18 +34,18 @@ Cubature integration rules are recommended (and the default).
| `Cylinder` | ✅ | 🛑 | ✅ |
| `CylinderSurface` | ✅ | ✅ | ✅ |
| `Disk` | ✅ | ✅ | ✅ |
| `Frustum` | 🛑 | 🛑 | 🛑 |
| `FrustumSurface` | 🛑 | 🛑 | 🛑 |
| `Frustum` | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/57) | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/57) | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/57) |
| `FrustumSurface` | ✅ | ✅ | ✅ |
| `Line` | ✅ | ✅ | ✅ |
| `ParaboloidSurface` | ✅ | ✅ | ✅ |
| `Plane` | ✅ | ✅ | ✅ |
| `Ray` | ✅ | ✅ | ✅ |
| `Ring` | ✅ | ✅ | ✅ |
| `Rope` | ✅ | ✅ | ✅ |
| `Segment` | ✅ | ✅ | ✅ |
| `SimpleMesh` | 🎗️ | 🎗️ | 🎗️ |
| `SimpleMesh` | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/27) | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/27) | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/27) |
| `Sphere` in `𝔼{2}` | ✅ | ✅ | ✅ |
| `Sphere` in `𝔼{3}` | ✅ | ✅ | ✅ |
| `Tetrahedron` in `𝔼{3}` | 🎗️ | ✅ | 🎗️ |
| `Tetrahedron` in `𝔼{3}` | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/40) | ✅ | [🎗️](https://github.com/mikeingold/MeshIntegrals.jl/issues/40) |
| `Triangle` | ✅ | ✅ | ✅ |
| `Torus` | ✅ | ✅ | ✅ |
4 changes: 2 additions & 2 deletions src/integral_aliases.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ function surfaceintegral(
Dim = Meshes.paramdim(geometry)

if Dim == 2
return integral(f, geometry, GaussKronrod())
return integral(f, geometry, HAdaptiveCubature())
else
error("Performing a surface integral on a geometry with $Dim parametric dimensions not supported.")
end
Expand Down Expand Up @@ -145,7 +145,7 @@ function volumeintegral(
Dim = Meshes.paramdim(geometry)

if Dim == 3
return integral(f, geometry, GaussKronrod())
return integral(f, geometry, HAdaptiveCubature())
else
error("Performing a volume integral on a geometry with $Dim parametric dimensions not supported.")
end
Expand Down
Loading