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

Dirchlet and Neumann aliases for Value and Gradient boundary conditions #355

Merged
merged 2 commits into from
Aug 13, 2019

Conversation

ali-ramadhan
Copy link
Member

Somehow I always find myself saying Dirchlet and Neumann in real-life so I'd like to use the names in the code.

Wish Value/Gradient was the accepted nomenclature...

@ali-ramadhan ali-ramadhan added the cleanup 🧹 Paying off technical debt label Aug 13, 2019
@codecov
Copy link

codecov bot commented Aug 13, 2019

Codecov Report

Merging #355 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #355   +/-   ##
=======================================
  Coverage   72.86%   72.86%           
=======================================
  Files          22       22           
  Lines        1157     1157           
=======================================
  Hits          843      843           
  Misses        314      314
Impacted Files Coverage Δ
src/boundary_conditions.jl 70% <ø> (ø) ⬆️

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 7a4142c...0e104c5. Read the comment docs.

@codecov
Copy link

codecov bot commented Aug 13, 2019

Codecov Report

Merging #355 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #355   +/-   ##
=======================================
  Coverage   72.86%   72.86%           
=======================================
  Files          22       22           
  Lines        1157     1157           
=======================================
  Hits          843      843           
  Misses        314      314
Impacted Files Coverage Δ
src/Oceananigans.jl 100% <ø> (ø) ⬆️
src/boundary_conditions.jl 70% <ø> (ø) ⬆️

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 7a4142c...cac07f2. Read the comment docs.

Copy link
Member

@glwagner glwagner left a comment

Choose a reason for hiding this comment

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

"Be the change you want to see in the world..."

Anyways, should we export these names?

@ali-ramadhan
Copy link
Member Author

True...

Feel free to merge. (I'm trying to not merge my own pull requests.)

@glwagner
Copy link
Member

Seems like good practice.

@glwagner glwagner merged commit 4c98658 into master Aug 13, 2019
@glwagner glwagner deleted the dirchlet-neumann branch August 13, 2019 13:18
arcavaliere pushed a commit to arcavaliere/Oceananigans.jl that referenced this pull request Nov 6, 2019
Dirchlet and Neumann aliases for Value and Gradient boundary conditions

Former-commit-id: 4c98658
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup 🧹 Paying off technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants