-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fixes for Open Boundary Systems for free surfaces #609
Fixes for Open Boundary Systems for free surfaces #609
Conversation
…bs_for_free_surface
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #609 +/- ##
==========================================
- Coverage 68.74% 68.71% -0.04%
==========================================
Files 86 86
Lines 5282 5299 +17
==========================================
+ Hits 3631 3641 +10
- Misses 1651 1658 +7
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
Conflicts: test/examples/examples.jl
src/schemes/boundary/open_boundary/method_of_characteristics.jl
Outdated
Show resolved
Hide resolved
src/schemes/boundary/open_boundary/method_of_characteristics.jl
Outdated
Show resolved
Hide resolved
Co-authored-by: Erik Faulhaber <[email protected]>
Co-authored-by: Erik Faulhaber <[email protected]>
Co-authored-by: Erik Faulhaber <[email protected]>
Co-authored-by: Erik Faulhaber <[email protected]>
Co-authored-by: Erik Faulhaber <[email protected]>
Co-authored-by: Erik Faulhaber <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the last conversation is solved.
* set test up for 1.11 * fix * typo * fix again * remove soundspeed from OBS * skip empty system * fix test * fix tests * fix tests * fix bug * fix * check dimensionality of reference functions * propagate characteristics * update * cleanup * update * Increase errors for 1.11 * Fix invalidations * Fix tests * Update ci.yml * revert * Update ci.yml * Update test/validation/validation.jl Co-authored-by: Erik Faulhaber <[email protected]> * revert changes that had no benefit * update * cleanup * include in test run * remove redundancy * revert * fix tests * fix * fix test * fix test * fix the test * fix error * Update src/schemes/boundary/open_boundary/method_of_characteristics.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update test/schemes/boundary/open_boundary/boundary_zone.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update src/setups/extrude_geometry.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update src/schemes/boundary/open_boundary/method_of_characteristics.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update src/schemes/boundary/open_boundary/boundary_zones.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update examples/fluid/pipe_flow_3d.jl Co-authored-by: Erik Faulhaber <[email protected]> * fix test * fix test * format * fix test * format --------- Co-authored-by: LasNikas <[email protected]> Co-authored-by: Erik Faulhaber <[email protected]>
* set test up for 1.11 * fix * typo * fix again * remove soundspeed from OBS * skip empty system * fix test * fix tests * fix tests * fix bug * fix * check dimensionality of reference functions * propagate characteristics * update * cleanup * update * Increase errors for 1.11 * Fix invalidations * Fix tests * Update ci.yml * revert * Update ci.yml * Update test/validation/validation.jl Co-authored-by: Erik Faulhaber <[email protected]> * revert changes that had no benefit * update * cleanup * include in test run * remove redundancy * revert * fix tests * fix * fix test * fix test * fix the test * fix error * Update src/schemes/boundary/open_boundary/method_of_characteristics.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update test/schemes/boundary/open_boundary/boundary_zone.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update src/setups/extrude_geometry.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update src/schemes/boundary/open_boundary/method_of_characteristics.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update src/schemes/boundary/open_boundary/boundary_zones.jl Co-authored-by: Erik Faulhaber <[email protected]> * Update examples/fluid/pipe_flow_3d.jl Co-authored-by: Erik Faulhaber <[email protected]> * fix test * fix test * format * fix test * format --------- Co-authored-by: LasNikas <[email protected]> Co-authored-by: Erik Faulhaber <[email protected]>
Depends on #610
Depends on #608
Depends on #606
Depends on #605
Depends on #603
Depends on #599