angelodlfrtr / open-meteo-rs Goto Github PK

Support to geo primitives like geo#Point could be added, maybe under a feature.

PR #5 introduced air-quality endpoint.

Some code is shared between the forecast and air-quality modules :

• extract_times, unix_time_to_naive_datetime methods.
• CellSelection, CurrentResult, ForecastResultHourly, ForecastResultItem structs.

Methods could be moved to an utils module and structs to a more generic form that could be shared with forecast and air-quality modules.

The code is not yet tested.

Tests should be added :

• Call endpoints, check responses types / values / params.
• TryFrom TryInto`, ... implementations.
• Utils methods
• ...

It seems that time_zone is sent, but timezone is required. See docs.

This is manifesting for me as this error

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: InvalidResponseStatus { status_code: 400, text: "{\"reason\":\"Timezone is required\",\"error\":true}" }', src/main.rs:21:43

When requesting daily data with slightly modified example code:

let mut opts = open_meteo_rs::forecast::Options::default();
opts.location = open_meteo_rs::Location {
    lat: 48.864716,
    lng: 2.349014,
};
opts.current_weather = Some(true);
opts.daily = vec![
    "temperature_2m_max".to_string(),
    "temperature_2m_min".to_string(),
    "precipitation_probability_max".to_string(),
    "windspeed_10m_max".to_string(),
];