Skip to content

Conversation

@jtneedels
Copy link
Contributor

Adds governing equations for NEMO solver (nonequilibrium NS and Euler) to theory.md. Work in conjunction with @WallyMaier.

Signed-off-by: jtneedels <jneedels@stanford.edu>
Signed-off-by: jtneedels <jneedels@stanford.edu>
Signed-off-by: jtneedels <jneedels@stanford.edu>
Copy link
Member

@pcarruscag pcarruscag left a comment

Choose a reason for hiding this comment

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

What is the .DS_Store file?


| Solver | Version |
| --- | --- |
| `NEMO_NAVIER_STOKES`, `NEMO_RANS` | 7.0.0 |
Copy link
Member

Choose a reason for hiding this comment

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

Is NEMO_RANS operational already?

Copy link
Contributor Author

@jtneedels jtneedels Jan 4, 2023

Choose a reason for hiding this comment

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

Last I talked to @WallyMaier it works in his branch, but it's not in develop. I think he is still doing V&V. I'll go ahead and remove it for now, and we can add it back in later when ready.

Copy link
Member

Choose a reason for hiding this comment

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

The documentation and code branches should be in sync (develop -> develop, master -> master)

@jtneedels
Copy link
Contributor Author

jtneedels commented Jan 4, 2023

What is the .DS_Store file?

Oops, didn't see that got added. Must be some kind of cache file for when I viewed the md as an html on my machine, I'll remove it.

Signed-off-by: jtneedels <jneedels@stanford.edu>
Signed-off-by: jtneedels <jneedels@stanford.edu>
@jtneedels jtneedels requested a review from pcarruscag January 4, 2023 20:19
@jtneedels jtneedels merged commit 0f2b2d1 into master Jan 4, 2023
@pr-triage pr-triage bot added the PR: merged label Jan 4, 2023
@jtneedels jtneedels deleted the feature_nemo_theory branch January 4, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants