Skip to content

Conversation

@mdumett
Copy link
Collaborator

@mdumett mdumett commented Feb 13, 2025

addBC functionality adds boundary information into the discrete analog of the PDE, both operator and rhs

addBC functionality adds boundary information into the discrete analog of the PDE, both operator and rhs
Copy link
Collaborator

@jbrzensk jbrzensk left a comment

Choose a reason for hiding this comment

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

Some of the codes use a interpolated that you probably just named different than the library.
Things that would make this perfect would be a 2D or 3D Neumann example, AND take an existing example, and show how to use the new boundary operators with it. Like how we do multiple version of wave equation, pick a varaiety and show how to use these new operators.

Corrected function names and added four examples: two are old ones written with addBC instead of robinBC, one 1D wave equation using addBC and one 3D with Neumann and periodic BC
Copy link
Collaborator

@jbrzensk jbrzensk left a comment

Choose a reason for hiding this comment

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

Works appropriately in both Octave and MATLAB. Examples work well. I have no other comments unless @aboada wants to chime in about the BC operators.

@aboada
Copy link
Collaborator

aboada commented Feb 21, 2025

Works appropriately in both Octave and MATLAB. Examples work well. I have no other comments unless @aboada wants to chime in about the BC operators.

Will need time to review. If somebody else can take a look at it that will be awesome 👍

@aboada
Copy link
Collaborator

aboada commented Feb 22, 2025

This PR adds a lot of files to the examples folder. Why don't we organize the examples into separate folders?
For instance, the Wave1D example can go to its own folder and have a readme file explaining the problem statement, explanations of the results, etc.

@mdumett
Copy link
Collaborator Author

mdumett commented Feb 22, 2025

Do you mean to organize all the examples (the ones that existed before and the ones contributed in this PR)? If that is the case, we could think organizing all examples by types, for example, elliptic, parabolic, hyperbolic, specific equations, etc.

@aboada
Copy link
Collaborator

aboada commented Feb 22, 2025

Do you mean to organize all the examples (the ones that existed before and the ones contributed in this PR)? If that is the case, we could think organizing all examples by types, for example, elliptic, parabolic, hyperbolic, specific equations, etc.

Yes, "organizing all examples by types, for example, elliptic, parabolic, hyperbolic, specific equations, etc." is what I meant.
Not in this PR, though. I would keep this PR focused on its purpose.

@mdumett mdumett merged commit 6c28ba0 into master Feb 22, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants