-
Notifications
You must be signed in to change notification settings - Fork 5
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
panic in conv_vec_to_value_u().unwrap #133
Comments
Yes, it seems that in line 249 the value of the clima sensor is Replace https://github.com/ElektraInitiative/opensesame/blob/177e14fd1ad29c04c1a838fc01f01b43150c5eae/src/clima_sensor_us.rs#L252C1-L256 with these lines. match conv_vec_to_value_s((response_temp[0], response_temp[1]) {
Ok(value) => {temp = value as 32 / 10.0;},
Err(_) => ()
}
match conv_vec_to_value_u((response_wind[0], response_wind[1]) {
Ok(value) => {wind = value as 32 / 10.0;},
Err(_) => ()
} According to the blog for Olimex, I will try to finish that in February. |
Thx for looking into it and creating the PR!
Sounds like a plan. 🚀 |
Maybe on too strong wind?
@fel115 do you have an idea what this could be? Any progress on the blog for Olimex?
The text was updated successfully, but these errors were encountered: