Skip to content

Conversation

@streeve
Copy link
Member

@streeve streeve commented Jun 1, 2023

Add new example. Use a new directory to differentiate between the tutorial for individual components

TODO: add a link in the wiki somewhere (proxy app page?)

@streeve streeve requested a review from kwitaechong June 1, 2023 16:47
@streeve streeve self-assigned this Jun 1, 2023
@streeve
Copy link
Member Author

streeve commented Jun 1, 2023

@colemanjs FYI

@streeve streeve force-pushed the finite_difference branch from 1877c5d to c2ea6ea Compare June 1, 2023 17:06
k - planes[b][2], 0 );
} );
}

Copy link
Collaborator

@kwitaechong kwitaechong Jun 1, 2023

Choose a reason for hiding this comment

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

It might be a good idea to add comment about the problem. I think this is about unsteady heat equation problem in 3d.

The governing equstion : $$\rho c_p dT/dt = k \nabla^2{T} + q$$
Neumann boundary condition (describing heat flux ) : $$k \nabla{T} \cdot n = f$$

Copy link

@colemanjs colemanjs Jun 8, 2023

Choose a reason for hiding this comment

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

@kwitaechong, you are correct. This boundary conditions is just an adiabatic one. I will correct the description.

Also, it is worth pointing out that I think the left-hand and right-hand sides are swapped...

Copy link
Member Author

Choose a reason for hiding this comment

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

@colemanjs can you add a more detailed comment below?

****************************************************************************/

#include <array>
#include <math.h>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
#include <math.h>
#include <cmath>

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

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