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

Wondering if the A Matrix is missing a (times) dt factor #2

Open
otmanon opened this issue Jul 27, 2020 · 0 comments
Open

Wondering if the A Matrix is missing a (times) dt factor #2

otmanon opened this issue Jul 27, 2020 · 0 comments

Comments

@otmanon
Copy link

otmanon commented Jul 27, 2020

Hey!
I'm implementing something similar following Bridson's textbook as well! Sometimes I look at this codebase as a reference ;)
Wondering however whether or not this is a mistake on my end or not. It seems the A Matrix as constructed in your pressureSolve() function has coefficients that are not multiplied by the timestep length, as the textbook suggests they should be.

Maybe you just assume a timestep of 1 and that works out for you?
Thanks

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