-
Notifications
You must be signed in to change notification settings - Fork 45
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
CI: Wind hindcast API Calls #315
Comments
The metocean example pulls wind toolkit data and is pretty fast. We should look at replacing our test data with what's used in that example |
I'm attempting to look into this today. I started testing locally but it seems like I'm already being rate limited. My initial testing indicates that multiparameter requests are running as expected (the metocean example only does this), but for some reason multiyear and multilocation requests are very slow. I'll keep investigating this today and next week as the NREL server allows |
Hopefully #315 shows some performance improvements. But in the typical manner of pulling this data, it was successful once locally and then downloads started hanging for long times. |
This PR addresses #315 by: - Changes the data called by the Wind Toolkit tests so that they run faster - Updates the .csv files that the tests compare against - Updates a few descriptions in the metocean example, fixes a sorting issue, reduces the data downloaded there - Tests in hindcast match the notebooks and use the same cache.
This PR addresses #315 by: - Changes the data called by the Wind Toolkit tests so that they run faster - Updates the .csv files that the tests compare against - Updates a few descriptions in the metocean example, fixes a sorting issue, reduces the data downloaded there - Tests in hindcast match the notebooks and use the same cache.
The wind hindcast data currently takes greater than 3.5 hours to collect the needed data.
Can we get this down to 1 hr to be on par with the wave data?
The text was updated successfully, but these errors were encountered: