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

Use bilinear interpolation method on v_grad for dvwind_ds in all coordinate systems #126

Closed
jhmatthews opened this issue Nov 13, 2014 · 2 comments
Labels

Comments

@jhmatthews
Copy link
Collaborator

In #118 I imposed a rather lazy fix for our incorrect calculation of dvds in spherical coordinates which meant that we now do an on the fly calculation of dvds in spherical coordinates only.

Ideally, we want to do an interpolation on v_grad here, which is stored at each cell vertex. This should be improved by figuring out out how the velocity gradient tensor ought to be rotated in order to give the right answer for spherical coordinates.

The mystery here is why changing the angle along which the grid is defined doesn't change the angle at which the answer is correct.

@jhmatthews jhmatthews added the low label Nov 13, 2014
@jhmatthews
Copy link
Collaborator Author

I'm closing this. Ideally we should figure it out, but it's not really a good use of time. The spherical model works with the on the fly methody, and it will never be a production model anyway.

@kslong
Copy link
Collaborator

kslong commented Jan 20, 2015

I believe you should redefine this as an enhancement. I think we are always going to need the spherical model. Knox

From: James Matthews <[email protected]mailto:[email protected]>
Reply-To: agnwinds/python <[email protected]mailto:[email protected]>
Date: Tuesday, January 20, 2015 at 10:43 AM
To: agnwinds/python <[email protected]mailto:[email protected]>
Subject: Re: [python] Use bilinear interpolation method on v_grad for dvwind_ds in all coordinate systems (#126)

I'm closing this. Ideally we should figure it out, but it's not really a good use of time. The spherical model works with the on the fly methody, and it will never be a production model anyway.


Reply to this email directly or view it on GitHubhttps://github.com/agnwinds/python/issues/126#issuecomment-70686762.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants