-
Notifications
You must be signed in to change notification settings - Fork 4
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
Python version of SWM code #2
Comments
Thanks @supreethms1809! We can take a look |
@supreethms1809 looks like initial elements of u are off |
Supreeth-
Happy to help too. I know SWM quite well. I have a Python version I created
for laughs soon after retiring.
Curious what tool you’re using to debug Python code, and why you created a
new Python SWM?
Rich
…On Wed, Feb 21, 2024 at 7:55 AM cenamiller ***@***.***> wrote:
@supreethms1809 <https://github.com/supreethms1809> looks like initial
elements of u are off
—
Reply to this email directly, view it on GitHub
<#2 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACPNOBQJV57POKBPK4XWBM3YUYDHBAVCNFSM6AAAAABDSKX5NWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJWHAZTENRYHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Yeah, the index has an issue when initializing the boundary conditions. So I have commented out the boundary conditions code. that's why the initial elements are off |
|
@cenamiller I have created a new Python version of the SWM code.
Currently, Boundary conditions are disabled. I am getting an index out-of-bound error when I enable it.
Need a little debugging.
The text was updated successfully, but these errors were encountered: