Skip to content
This repository was archived by the owner on Jun 21, 2024. It is now read-only.

Rain visualisation - #9

Open
spasutto wants to merge 3 commits into
vicb:masterfrom
spasutto:master
Open

Rain visualisation#9
spasutto wants to merge 3 commits into
vicb:masterfrom
spasutto:master

Conversation

@spasutto

@spasutto spasutto commented Jun 2, 2023

Copy link
Copy Markdown

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

@spasutto

spasutto commented Jun 2, 2023

Copy link
Copy Markdown
Author

Avec convection
cumulus
Sans convection
nocumulus

Comment thread src/selectors/skewt.js
});

params.level = levels;
params.snow = false;

Copy link
Copy Markdown
Owner

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?

Comment thread src/selectors/skewt.js
params.level = levels;
params.snow = false;
params.rain = false;
const nextfc = forecasts.forecast.data.ts.findIndex((t) => t >= timestamp);

Copy link
Copy Markdown
Owner

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?

Copy link
Copy Markdown
Author

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 !

Comment thread src/components/parcel.js Outdated
import { h } from "preact";

export function Parcel({ parcel, width, line, pToPx, formatAltitude }) {
export function Parcel({ parcel, width, line, pToPx, formatAltitude, params }) {

Copy link
Copy Markdown
Owner

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?

@vicb vicb left a comment

Copy link
Copy Markdown
Owner

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.

@spasutto

spasutto commented Jun 6, 2023

Copy link
Copy Markdown
Author

J'ai corrigé.
C'est très perso mais sur mon mobile le plugin prend toute la place visible et du coup dans mon cas d'utilisation ça double le temps pour me faire une idée de la journée :

  • d'abord sur la carte je 'scroll' toute la journée pour voir s'il y'a des précipitations
  • ensuite je vérifie les emagrammes avec le plugin

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 ...
Mais encore une fois pas de soucis si c'est tu trouves que c'est pas le but du plugin 👍

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

@vicb

vicb commented Jun 6, 2023

Copy link
Copy Markdown
Owner

Windy a fait une grosse release hier.
Je viens seulement de publier mes changements ca va t'aider pour le dev.

J'aime bien l'idee de ton update et je suis pour si c'est utile.
Par contre j'essaye de faire as simple as simple pour le code et l'UI.
Du coup que penses tu de toujours afficher la goutte aux meme endroit i.e. a gauche avec les nuages?

@spasutto

spasutto commented Jun 7, 2023

Copy link
Copy Markdown
Author

Ok, toujours en position fixe comme ça ?
image image

Comment thread src/selectors/skewt.js
params.level = levels;
params.snow = false;
params.rain = false;
const nextfc = forecasts.forecast.data.ts.findIndex((t) => t >= timestamp);

Copy link
Copy Markdown
Owner

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)

@vicb

vicb commented Jun 7, 2023

Copy link
Copy Markdown
Owner

Yep...

J'ai pense un peu plus a ta modif et j'aime bien.

Ce que je regarde souvent:

  • la hauteur de la pluie en mm,
  • la couverture nuageuse en %

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants