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

Update meteo stations #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update meteo stations #8

wants to merge 1 commit into from

Conversation

f-baerenbold
Copy link
Contributor

You used an older version of Lakes.json so I had to update quite a few of the stations. This is by no means a definite file. Maybe a closer look at all meteo stations used (and potential alternatives) could make sense. However, it seems to me that the changes induced by changing the meteo station for a simulation were not that big. Hence, I often opted for a meteo station with a longer data record (e.g. from Thun to Interlaken and from Egolzwil to Lucerne).

@JamesRunnalls
Copy link
Contributor

Seems reasonable, I will need to recalibrate the lakes before merging the request. Do you forsee any additional edits to the file that would mean it could make sense to wait to recalibrate?

@f-baerenbold
Copy link
Contributor Author

I don't think I will have time to do any further work on this this year. Maybe I can check more things next year. So from my side you can do a recalibration but it is also not urgent because I am not currently using the results of the simulations.

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 this pull request may close these issues.

2 participants