-
Notifications
You must be signed in to change notification settings - Fork 5
Rain visualisation #9
base: master
Are you sure you want to change the base?
Conversation
@@ -59,6 +59,15 @@ export const params = createSelector(forecasts, timestamp, pMin, (forecasts, tim | |||
}); | |||
|
|||
params.level = levels; | |||
params.snow = false; |
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.
A supprime si pas utilise?
@@ -59,6 +59,15 @@ export const params = createSelector(forecasts, timestamp, pMin, (forecasts, tim | |||
}); | |||
|
|||
params.level = levels; | |||
params.snow = false; | |||
params.rain = false; | |||
const nextfc = forecasts.forecast.data.ts.findIndex((t) => t >= timestamp); |
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.
Cette ligne va dans le if en dessous, right?
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.
oups! Le if teste nextfc et pas next !
src/components/parcel.js
Outdated
@@ -2,7 +2,7 @@ import { PureComponent } from "./pure"; | |||
// eslint-disable-next-line no-unused-vars | |||
import { h } from "preact"; | |||
|
|||
export function Parcel({ parcel, width, line, pToPx, formatAltitude }) { | |||
export function Parcel({ parcel, width, line, pToPx, formatAltitude, params }) { |
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.
tu pourrais directement passer rain au lieu de params?
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.
Merci pour la PR.
Tu trouves que ca aide?
J'ai ajoute qqes commentaires.
J'ai corrigé.
Et je me suis dit que le plugin affichait déjà des infos sur la la base des cumulus et que cette info n'était pas visuellement très lourde ... PS : je sais pas pourquoi ce matin je n'arrive plus à faire tourner le rollup de dev, que ce soit avec ou sans mes deux commits |
Windy a fait une grosse release hier. J'aime bien l'idee de ton update et je suis pour si c'est utile. |
@@ -59,6 +59,14 @@ export const params = createSelector(forecasts, timestamp, pMin, (forecasts, tim | |||
}); | |||
|
|||
params.level = levels; | |||
params.snow = false; | |||
params.rain = false; | |||
const nextfc = forecasts.forecast.data.ts.findIndex((t) => t >= timestamp); |
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.
Je pense que tu peux utiliser sampleAt()
plutot si la la valeur de rain est numerique (vs bool)
Yep... J'ai pense un peu plus a ta modif et j'aime bien. Ce que je regarde souvent:
Je pense que ce serait d'ajouter ces infos. Pour la pluie la hauteur me semble plus pertinente qu'un on/off. Tu peux demander sur le forum de windy comment trouver cette hauteur en mm ils sont sympas et aident bien. Pour l'affichage, j'imagine bien icon outline + valeur, plus gros & grises en background du sounding. Si tu veux en discuter de vive voix tu peux m'envoyer un mail (email de mon profil) et on arrange une visio. |
Salut,
J'utilise pour chaque vol ton super plugin sur mon mobile, et pour gagner du temps en plus de voir rapidement le vent et l'instabilité/plafonds, j'aimerais avoir une visu rapide de la pluie (des fois y'a pas de vent et de supers plafs mais en fait ... il pleut).
Je connaissais pas l'API Windy avant de bricoler, et j'ai pas vraiment trouvé la doc sur les différentes valeurs remontées par l'api PointForecast. J'ai vu qu'elle a un tableau "rain" qui ressemble à un booléen, je me suis appuyé dessus pour afficher ou non une goutte d'eau sur le cumulus ou dans la couche supérieur si pas de convection.
Je sais pas si c'est pas naïf comme vision de la chose alors n'hésite pas à me dire si c'est utilisable
@+
Sylvain