-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add iotools functions for Meteonorm #2499
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor things here and there. Otherwise LGTM!
Specification of the hoirzon line. Can be either 'flat' or 'auto', or | ||
specified as a list of 360 horizon elevation angles. The default is | ||
'auto'. | ||
terrain: string, optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
terrain: string, optional | |
terrain: str, optional |
with the same random seed will yield identical results. | ||
clear_sky_radiation_model : {'esra', 'solis'} | ||
Which clearsky model to use. The default is 'esra'. | ||
data_version : string, optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
data_version : string, optional | |
data_version : str, optional |
data_version : string, optional | ||
Version of Meteonorm climatological data to be used. The default is | ||
'latest'. | ||
future_scenario: string, optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
future_scenario: string, optional | |
future_scenario: str, optional |
'latest'. | ||
future_scenario: string, optional | ||
Future climate scenario. | ||
future_year : integer, optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
future_year : integer, optional | |
future_year : int, optional |
where applicable. See variable :const:`VARIABLE_MAP`. The default is | ||
True. | ||
url: str, optional. | ||
Base URL of the Meteonorm API. 'climate/tmy'` is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Base URL of the Meteonorm API. 'climate/tmy'` is | |
Base URL of the Meteonorm API. `'climate/tmy'` is |
In decimal degrees, east is positive (ISO 19115). | ||
api_key: str | ||
Meteonorm API key. | ||
parameters: list, optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this should be list or 'all' (like the turbidity)?
(north=0, east=90, south=180, west=270). The default is 180. | ||
time_step: {'1min', '1h'}, optional | ||
Frequency of the time series. The default is '1h'. | ||
horizon: optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
horizon: optional | |
horizon: str, optional |
time_step : {'1min', '15min', '1h'}, optional | ||
Frequency of the time series. The parameter is ignored when requesting | ||
forcasting data. The default is '15min'. | ||
horizon : optional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
horizon : optional | |
horizon : str, optional |
docs/sphinx/source/reference
for API changes.docs/sphinx/source/whatsnew
for all changes. Includes link to the GitHub Issue with:issue:`num`
or this Pull Request with:pull:`num`
. Includes contributor name and/or GitHub username (link with:ghuser:`user`
).remote-data
) and Milestone are assigned to the Pull Request and linked Issue.This PR adds functions for retrieving irradiance and weather data from Meteonorm. This is the last function I plan on contributing to the commercial iotools, as pvlib then will support what I consider the major providers of satellite-derived irradiance data.
Documentation: