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

allow different theta for wall motion and dP/ds, bulk vel #7

Open
johnfgibson opened this issue Nov 9, 2018 · 1 comment
Open

allow different theta for wall motion and dP/ds, bulk vel #7

johnfgibson opened this issue Nov 9, 2018 · 1 comment

Comments

@johnfgibson
Copy link
Collaborator

johnfgibson commented Nov 9, 2018

Is your feature request related to a problem? Please describe.
We currently have one angle, theta, for changing the orientation of the wall velocity and imposed pressure gradient or bulk velocity relative to the computational box. However, there's no reason these shouldn't be independent.

Describe the solution you'd like
I propose splitting -theta into -thetaWall and -thetaBulk, and applying them as follows in dnsflags.cpp

    ulowerwall = -Uwall_ * cos(thetaWall_);
    uupperwall = Uwall_ * cos(thetaWall_);
    wlowerwall = -Uwall_ * sin(thetaWall_);
    wupperwall = Uwall_ * sin(thetaWall_);
    Vsuck = Vsuck_;
    rotation = rotation_;
    dPdx = dPds_ * cos(thetaBulk_);
    dPdz = dPds_ * sin(thetaBulk_);
    Ubulk = Ubulk_ * cos(thetaBulk_);
    Wbulk = Ubulk_ * sin(thetaBulk_);

This issue came up as I started documenting the base/fluctuation decomposition in the channelflow-2 paper. I have a feeling that our nomenclature and input parameters have evolved organically from a simple unidirectional beginning, and will end up needing some revisions to really clarfiy the relation of the input parameters to the math.

Perhaps also we could allow for different frames of reference with respect to moving walls. I found myself wanting this for ASBL, to have a lower wall with speed 0 at y=0 and an upper wall with speed 1 at y=H. Granted this is transformation is pretty trivial, but I think it would still be nice for direct comparisons between near-wall dynamics between channel and ASBL.

Comments?

@johnfgibson johnfgibson changed the title allow differnet theta for wall motion and dP/ds, bulk vel allow different theta for wall motion and dP/ds, bulk vel Nov 10, 2018
@johnfgibson
Copy link
Collaborator Author

It was complicated enough to figure out and implement the general laminar solution to arbitrary combinations of wall speeds, positions, suction velocity, pressure gradient and bulk constraint, assuming alignment of wall motion and pressure gradient (#9). Adding this generalization might not be that much more complicated, but I think I'll let someone who needs it do it.

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

No branches or pull requests

1 participant