-
Notifications
You must be signed in to change notification settings - Fork 53
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
code error in helpers/genNetCDF/Topography.py #180
Comments
Hi @wwwliye thanks for filing this issue. Q1: I think @scrasmussen assigned himself to fix this, it does look like there is an inconsistency there. Q2: ICAR does not read in the "w" field, so it shouldn't be an issue. At one point, ICAR had an option to interpolate, e.g. Z and P if they were on interface levels instead of mass levels, though I think that functionality may have been dropped with the move to v2. I recommend having all forcing data on the same levels. Q3: I am not sure, this may have used a different topographic hill than in Christoph Schar's paper, I'm not sure if @scrasmussen or @bertjebertjek are best to answer this one. |
Thank you for pointing this out! It will be fixed in an upcoming PR.
In the paper, on pdf page 7 and page 2465 is the following diagram: This should match up to the generated file? I'll ping the person who wrote this to double check my interpretation is correct, @bertjebertjek. |
Hey, sorry for the late reply. |
question1: in Topography.py, in function "genHill": j = (np.arange(self.nx) - self.ny/2) / self.ny * np.pi *2, is this nx should be ny?
question2: in icar, if all variables's levels must be same? in wrfout data , vertical velocity w's levels are u's levels plus 1,
when running icar, there will be some errors.
question3: also in Topography.py, is Schar's advection test's code correct?, in Schar's article in 2004, I don't found the same formula ,especially "hgt = (
hill_height *
self.h_x(ig, lmbda) * self.h_x_star( ig, a)
)" , that make me very confused
hope your reply, thank you !
The text was updated successfully, but these errors were encountered: