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

the EVAP variable is always zero in BMI #114

Closed
MostafaGomaa93 opened this issue Nov 28, 2024 · 3 comments · Fixed by #116
Closed

the EVAP variable is always zero in BMI #114

MostafaGomaa93 opened this issue Nov 28, 2024 · 3 comments · Fixed by #116

Comments

@MostafaGomaa93
Copy link
Contributor

MostafaGomaa93 commented Nov 28, 2024

The Èvap variable is always zero through BMI, although it has a value. I checked that if we delete the all_timesteps = True in this line, the Èvap variable will get its value in BMI

This was referenced Nov 28, 2024
@BSchilperoort
Copy link
Contributor

BSchilperoort commented Nov 29, 2024

The all_timesteps variable denotes if the variable is an array which contains data on all time steps. This bug probably crept in because EVAP is an array of size n_timesteps, but only the first index is used to put any value in.

Can you check this and open an issue in STEMMUS_SCOPE? This could lead to incorrect results if sometimes it's used as EVAP[index_time] = x and sometimes EVAP[0] = x...

@SarahAlidoost
Copy link
Member

SarahAlidoost commented Dec 5, 2024

@MostafaGomaa93 Note that there are two variables EVAP and Evap 😄 . The line you mentioned is related to EVAP and not Evap. These two variables have a similar problem to variable Trap, see this comment. Can you also fix the code to have one variable for evapotranspiration?

@MostafaGomaa93
Copy link
Contributor Author

@MostafaGomaa93 Note that there are two variables EVAP and Evap 😄 . The line you mentioned is related to EVAP and not Evap. These two variables have a similar problem to variable Trap, see this comment. Can you also fix the code to have one variable for evapotranspiration?

Okay, I removed EVAP and adjusted Evap in this PR

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

Successfully merging a pull request may close this issue.

3 participants