-
Notifications
You must be signed in to change notification settings - Fork 62
Variable NS_High in fastfarm.py is not computed correctly #62
Comments
Thanks for your report. I'm surprised that In #56, @rthedin is doing a rewrite of the case creation. Maybe he has a comment on it? |
The choice of variable names is unfortunate here. The variables
When creating the high-resolution boxes, we do use the number of grid points and add the +1 as given by the documentation: python-toolbox/pyFAST/fastfarm/fastfarm.py Lines 463 to 465 in 5665bfb
Similarly, for the low-resolution box: python-toolbox/pyFAST/fastfarm/fastfarm.py Lines 435 to 437 in 5665bfb
Two comments:
|
Thank you for your comments. It appears it was confusion on my part and lucky coincidences. Although I have some questions.
A quick inspection shows that when you clone the branch the VERSION file comes with "0.1.0" written and if I change it to 3.3.0 as is in the branch it installs successfully. So I don't know what's going on there. Thanks for your time! |
Regarding 1. I think I managed to fix my inconsistencies and now its working properly. |
Hello everyone,
In the past few days, I've been trying to setup a FAST.Farm case using this toolbox. However I had a domain issue which made my FAST.Farm simulations fail.
This error was:
After looking through the FAST.Farm documentation NS_high is computed as such, with S : X,Y,Z
While in the fastfarm.py file it is defined as such
nX_High = int(np.ceil(LX_High/dX_High))
nY_High = int(np.ceil(LY_High/dY_High))
nZ_High = int(np.ceil(LZ_High/dZ_High))
As you can see the +1 is missing. The moment I added it the FAST.Farm simulations started working.
Thanks you!
The text was updated successfully, but these errors were encountered: