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

Adding temperature to PVade via Rayleigh-Benard convective flow example #52

Open
wants to merge 24 commits into
base: dev
Choose a base branch
from

Conversation

brookeslawski
Copy link
Collaborator

@brookeslawski brookeslawski commented Feb 2, 2024

Addressing Issue #46

New changes applied while adding temperature:

  • Changed forked branch to a new one created from upstream/sync
  • Replaced geom_dim with ndim to fix 2d/3d issues and added special treatment for ndim==2 versus nimd==3 where necessary (which explains the changes to the structure files)

@brookeslawski brookeslawski marked this pull request as draft February 2, 2024 22:13
@arswalid
Copy link
Collaborator

  • cleanup the code show results and PR should be ready

@brookeslawski brookeslawski marked this pull request as ready for review February 20, 2024 23:36
@brookeslawski
Copy link
Collaborator Author

Regarding "This branch is out-of-date with the base branch" - I'm not sure why this is because I ran a git pull upstream dev before pushing so the dev branch's latest changes should be in there.

@brookeslawski brookeslawski changed the title Adding temperature to PVade via Rayleigh-Benard convective flow example Adding air temperature to PVade via a heated panels example Nov 12, 2024
@brookeslawski brookeslawski changed the title Adding air temperature to PVade via a heated panels example Adding temperature to PVade via Rayleigh-Benard convective flow example Nov 12, 2024
@brookeslawski
Copy link
Collaborator Author

Planning to close this PR without merging as it has been superseded by PR #73

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.

3 participants