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

[BUG] Channel flow with free stream velocity correction #194

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anastazja-virtonomy
Copy link
Contributor

FreeStreamVelocityCorrection is causing unexpected changes in the density distribution and incorrect velocity profile near the outlet

@Xiangyu-Hu
Copy link
Owner

How long is the runtime of this test?

@anastazja-virtonomy
Copy link
Contributor Author

The test is set for 200 sec. of physical time, but the flow disturbance is visible already after 1-2 sec. which took around 1 min to simulate on my PC

@Xiangyu-Hu
Copy link
Owner

The test is set for 200 sec. of physical time, but the flow disturbance is visible already after 1-2 sec. which took around 1 min to simulate on my PC

Does problem solved?

@YuVirtonomy
Copy link
Collaborator

The test is set for 200 sec. of physical time, but the flow disturbance is visible already after 1-2 sec. which took around 1 min to simulate on my PC

Does problem solved?

This problem is solved.

  • Need to add TimeDependentAcceleration
  • Should not use parabolic flow for the inlet but uniform flow.
    I think this pull/issue should be closed

@Xiangyu-Hu
Copy link
Owner

The test is set for 200 sec. of physical time, but the flow disturbance is visible already after 1-2 sec. which took around 1 min to simulate on my PC

Does problem solved?

This problem is solved.

  • Need to add TimeDependentAcceleration
  • Should not use parabolic flow for the inlet but uniform flow.
    I think this pull/issue should be closed

Could you make a ci test in the pull request ao that we can merge it to master. Thanks.

@Xiangyu-Hu Xiangyu-Hu linked an issue Mar 1, 2023 that may be closed by this pull request
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.

FreeStreamVelocityCorrection not working properly in 3D
3 participants