nghyd issueshttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues2023-08-29T17:31:16+02:00https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/626Documentation lois d'ouvrages: ajouter les références des limites de coef d'e...2023-08-29T17:31:16+02:00Dorchies DavidDocumentation lois d'ouvrages: ajouter les références des limites de coef d'ennoiementMail du 24/07/2023 de Sylvain Richard:
> Par rapport aux messages d’avertissement de Cassiopée, je partage l’intérêt de modifier le message actuel relatif à l’équation de Villemonte, en retenant la valeur de 90% comme ennoiement maximal...Mail du 24/07/2023 de Sylvain Richard:
> Par rapport aux messages d’avertissement de Cassiopée, je partage l’intérêt de modifier le message actuel relatif à l’équation de Villemonte, en retenant la valeur de 90% comme ennoiement maximal, de manière à être en cohérence avec la publi de Villemonte et les guides d’hydraulique.
>
> Pour les autres formules, il n’y a pas toujours d’infos précises sur les conditions d’ennoiement dans la littérature, mais je partage la nécessité de mieux préciser les messages renvoyés par Cassiopée et de mettre qq explications pour chaque formules.Version 4.18.0Dorchies DavidDorchies Davidhttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/625L'application Electron sous Windows ne fonctionne plus2023-08-29T17:16:39+02:00Dorchies DavidL'application Electron sous Windows ne fonctionne plusTesté avec le v4.17.0. L'installation et l'utilisation au premier chargement fonctionne bien mais les lancements suivants donnent un écran vide:
![image](/uploads/f10eedd2c9cf0e6e4ccb77cb665cb0ec/image.png)Testé avec le v4.17.0. L'installation et l'utilisation au premier chargement fonctionne bien mais les lancements suivants donnent un écran vide:
![image](/uploads/f10eedd2c9cf0e6e4ccb77cb665cb0ec/image.png)Version 4.18.0https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/624L'URL https://cassiopee.g-eau.fr/cassiopee-releases/ est accessible aléatoire...2023-08-29T16:36:39+02:00Dorchies DavidL'URL https://cassiopee.g-eau.fr/cassiopee-releases/ est accessible aléatoirement## Symptômes
L'adresse https://cassiopee.g-eau.fr/cassiopee-releases/ renvoie sur l'application Cassiopée la plupart du temps.
L'adresse https://cassiopee.g-eau.fr/cassiopee-releases peut fonctionner. Une fois qu'une de ces deux adress...## Symptômes
L'adresse https://cassiopee.g-eau.fr/cassiopee-releases/ renvoie sur l'application Cassiopée la plupart du temps.
L'adresse https://cassiopee.g-eau.fr/cassiopee-releases peut fonctionner. Une fois qu'une de ces deux adresses a fonctionné dans le navigateur ouvert (testé sur Firefox), l'URL fonctionne jusqu'à fermeture de la session du navigateur.
Taper l'adresse complète fonctionne toujours: https://cassiopee.g-eau.fr/cassiopee-releases/index.php
## Hypothèses d'origine du bug
Je pense que c'est lié au Service worker qui met l'application en cache dans le navigateur, qui n'effectue pas de requête vers le serveur pour récupérer la page et qui retourne sur la page à la racine du web worker car cette page n'est pas stockée dans le cache.
D'après https://stackoverflow.com/questions/47693093/angular-5-and-service-worker-how-to-exclude-a-particular-path-from-ngsw-config/58520498#58520498
La question de bypass du service worker sur un dossier devrait être traité par la directive https://angular.io/guide/service-worker-config#navigationurls
Ce devrait pouvoir se résoudre en ajoutant la directive suivante dans le fichier https://gitlab.irstea.fr/cassiopee/nghyd/-/blob/devel/ngsw-config-template.json
```json
"navigationUrls": [
"!/cassiopee_releases"
]
```
A noter que d'après le lien so ci-dessus, la syntaxe `!` de négation ne fonctionne pas dans la directive `"ressources"`, or on l'utilise actuellement dans notre configuration.Version 4.18.0Dorchies DavidDorchies Davidhttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/622Mise à jour vers Angular 162023-06-07T16:30:56+02:00Grand FrancoisMise à jour vers Angular 16Version 4.18.0Grand FrancoisGrand Francoishttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/613Afficher la raison de la désactivation du bouton Calculer2023-03-20T14:08:37+01:00Grand FrancoisAfficher la raison de la désactivation du bouton CalculerQuand dans un formulaire le bouton Calculer peut être dégrisé pour une raison quelquonque (voire plusieurs), il faudrait afficher (sous le bouton lui même par exemple) un message expliquant pourquoi le calcul n'est pas possible.Quand dans un formulaire le bouton Calculer peut être dégrisé pour une raison quelquonque (voire plusieurs), il faudrait afficher (sous le bouton lui même par exemple) un message expliquant pourquoi le calcul n'est pas possible.https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/609Ajout du module courbe de remous d'une passe à macro-rugosité2023-06-01T08:43:10+02:00Dorchies DavidAjout du module courbe de remous d'une passe à macro-rugositéCe module sert, comme son nom l'indique, à calculer une courbe de remous dans une passe à macro-rugosité simple (un module de type `MacroRugo` pas une PAM complexe). Le principe consiste à calculer la courbe de remous à partir des caract...Ce module sert, comme son nom l'indique, à calculer une courbe de remous dans une passe à macro-rugosité simple (un module de type `MacroRugo` pas une PAM complexe). Le principe consiste à calculer la courbe de remous à partir des caractéristiques géométriques de la passe et du "Strickler équivalent" calculé dans la passe pour une hauteur d'eau donnée. A chaque point de la courbe de remous, on calcule la pente de la passe avec la profondeur calculée au point précédent ce qui revient à calculer la pente de la courbe de remous en ce point. De ce calcul, on récupère un Stricker équivent qu'on utilise dans le calcul de la courbe de remous pour calculer le point suivant.
- [x] Ajouter un bouton "Courbe de remous dans une passe" dans le bloc des Passes à macro-rugosités (PAM).
Le module de calcul hérite du module "Courbe de remous" à la différence que la forme de la section et ses caractéristiques géométriques le Strickler et le débit sont imposées par la PAM.
A la place du choix de la section et de ses paramètres, on choisira donc parmi les PAM disponibles dans la session (comme dans le vérificateur mais pour les PAM uniquement).
- [x] Ajouter un bouton "Générer une courbe de remous" dans le module PAM qui ouvre un module "Courbe de remous dans une passe" en présélectionnant la PAM d'origine.
- [x] Créer un module pour la classe `MacroRugoRemous` avec les champs suivants:
- Une liste déroulante des passes à macro-rugosités simples
- La cote de l'eau aval (`Z2`)
- La méthode de résolution
- Le pas de discrétisation
- Le choix de la donnée à calculer
Le calcul est uniquement fluvial. On ne saisira donc pas de cote de l'eau amont qui devrait être facultative dans la courbe de remous (réglé par cassiopee/nghyd#610).Version 4.18.0Grand FrancoisGrand Francoishttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/600Vérifier la remise à zéro des résultats sur changement de valeur de select2023-02-08T07:49:42+01:00Grand FrancoisVérifier la remise à zéro des résultats sur changement de valeur de selectDans un test e2e.
Une fois la migration vers autre chose que Protractor faite.
Et quand le monde sera devenu un paradis (ou avant).Dans un test e2e.
Une fois la migration vers autre chose que Protractor faite.
Et quand le monde sera devenu un paradis (ou avant).Grand FrancoisGrand Francoishttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/599Tests : la correction d'un bug dans les fonctions utilitaires e2e ne change p...2023-02-06T14:51:24+01:00Grand FrancoisTests : la correction d'un bug dans les fonctions utilitaires e2e ne change pas le comportementDans la fonction `setParamMode()` (`calculator.po`), une correction a été apportée pour valider (plutôt qu'annuler) le dialogue s'ouvrant quand on passe un paramètre en mode varié.
Cela ne change pas le résultat des tests, ce qui semble...Dans la fonction `setParamMode()` (`calculator.po`), une correction a été apportée pour valider (plutôt qu'annuler) le dialogue s'ouvrant quand on passe un paramètre en mode varié.
Cela ne change pas le résultat des tests, ce qui semble suspect : en effet, si on modifie un paramètre, c'est qu'on s'attend à un effet sur le résultat du test...
Vérifier ce qu'il en est pour tous les cas où on utilise `setParamMode(xxx, "var")`.Grand FrancoisGrand Francoishttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/594Grilles: ajouter les schémas de profil issus de Lemkecher et al. (2020)2023-08-29T17:25:28+02:00Dorchies DavidGrilles: ajouter les schémas de profil issus de Lemkecher et al. (2020)En attente des schémas a réaliser par @dcourret pour les intégrés dans le tableau des coefficients de profils sur le même modèle que les coefficient de trainée associés à la forme des plots dans les passes macrorugo.En attente des schémas a réaliser par @dcourret pour les intégrés dans le tableau des coefficients de profils sur le même modèle que les coefficient de trainée associés à la forme des plots dans les passes macrorugo.Version 4.18.0https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/591MacroRugo: Mise à jour de la formule f(h*) à partir de Cassan 20232023-05-12T10:40:24+02:00Dorchies DavidMacroRugo: Mise à jour de la formule f(h*) à partir de Cassan 2023Mise à jour de la documentation.Mise à jour de la documentation.Version 4.18.0Dorchies DavidDorchies Davidhttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/621Utiliser des identifiants symboliques plutôt que numériques dans les fichiers2023-06-01T08:43:18+02:00Grand FrancoisUtiliser des identifiants symboliques plutôt que numériques dans les fichiersDans certains fichiers (de session et peut être d'autres) et dans les tests Jasmine et/ou e2e, la valeur du champ calcType est numérique. Cela peut poser un problème de compatibilité ascendante.
Il faudrait s'assurer que des valeurs tex...Dans certains fichiers (de session et peut être d'autres) et dans les tests Jasmine et/ou e2e, la valeur du champ calcType est numérique. Cela peut poser un problème de compatibilité ascendante.
Il faudrait s'assurer que des valeurs texte sont utilisées à la place (par exemple en convertissant d'anciens fichiers exemple de session, ...).
Cette conversion est peut être nécessaire sur d'autres champs (loi de débit, type de structure, loi de perte de charge, ...), cad ceux codés par des enums.Version 4.18.0Grand FrancoisGrand Francoishttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/525Electron: Le zoom n'est pas disponible2023-03-03T16:55:15+01:00Dorchies DavidElectron: Le zoom n'est pas disponibleVoir dans les options de l'appli créée sous Electron comment activer le zoom par Control+Molette.Voir dans les options de l'appli créée sous Electron comment activer le zoom par Control+Molette.https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/521Résultats de calcul: Afficher les nombres entiers comme des nombres entiers2022-04-06T12:08:31+02:00Dorchies DavidRésultats de calcul: Afficher les nombres entiers comme des nombres entiersCela revient à rajouter une spécification sur les variables de résultat pour indiquer que ce sont des nombres entiers et qu'il faut utiliser un traitement d'affichage différent des autres paramètres/résultats.
Sachant que les résultats ...Cela revient à rajouter une spécification sur les variables de résultat pour indiquer que ce sont des nombres entiers et qu'il faut utiliser un traitement d'affichage différent des autres paramètres/résultats.
Sachant que les résultats affichent des paramètres d'entrée, des paramètres calculés et des résultats complémentaires, je ne sais pas trop où on peut spécifier cela...https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/512Documentation > Lois d'ouvrages: homogénéisér schémas, formules et nom des lois2023-10-09T09:30:37+02:00Dorchies DavidDocumentation > Lois d'ouvrages: homogénéisér schémas, formules et nom des lois- Nommer correctement les lois Cemagref dans l'interface et la doc:
- Seuil épais (Cemagref-D)
- Seuil épais (Cemagref-V)
- Seuil épais / Orifice (Cemagref-D)
- Seuil épais / Vanne de fond (Cemagref-V)
- Intégrer un schéma du pro...- Nommer correctement les lois Cemagref dans l'interface et la doc:
- Seuil épais (Cemagref-D)
- Seuil épais (Cemagref-V)
- Seuil épais / Orifice (Cemagref-D)
- Seuil épais / Vanne de fond (Cemagref-V)
- Intégrer un schéma du profil des déversoirs triangulaires et ½ triangulaires tronqués
- Ajouter une définition de la relation entre cote et "charge" ($`h_1 = z_1 - z_s`$...) dans la documentation avec un lien sur chaque formule.
- Vérifier les noms de variables des formules et les homogénéiser (ex: L et b pour la largeur)Version 4.18.0Dorchies DavidDorchies Davidhttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/506Lois d'ouvrages: Import/export des ouvrages depuis et vers SIC2022-02-11T15:05:21+01:00Dorchies DavidLois d'ouvrages: Import/export des ouvrages depuis et vers SIChttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/491Publish on cassiopee.g-eau.fr the TypeDoc pages of jalhyd2022-05-27T18:24:04+02:00Dorchies DavidPublish on cassiopee.g-eau.fr the TypeDoc pages of jalhydMaybe on https://cassiopee.g-eau.fr/jalhydMaybe on https://cassiopee.g-eau.fr/jalhydhttps://gitlab.irstea.fr/cassiopee/nghyd/-/issues/473Stopper un calcul trop long2020-09-28T09:39:21+02:00Mathias ChouetStopper un calcul trop longParfois un calcul peut être très long : Solveur, Prébarrages… Le navigateur mouline, mouline, et finit en général par proposer d'arrêter le script.
Oui mais, s'il ne le fait pas (vieux navigateur ?), ou si l'utilisateur appuie sur des c...Parfois un calcul peut être très long : Solveur, Prébarrages… Le navigateur mouline, mouline, et finit en général par proposer d'arrêter le script.
Oui mais, s'il ne le fait pas (vieux navigateur ?), ou si l'utilisateur appuie sur des commandes (ex: page précédente) avant d'arrêter le script, ou si l'utilisateur ferme l'onglet et perd son travail ? Il serait préférable d'avoir un mécanisme pour arrêter un calcul.
Oui mais, Javascript est monothread alors comment stopper un morceau de code exécuté dans le navigateur ?
* utiliser des WebWorkers, pardi : https://medium.com/jspoint/achieving-parallelism-in-javascript-using-web-workers-8f921f2d26db
On pourrait aussi déléguer le calcul à un serveur appelé via un webservice, mais cela revient à changer toute l'architecture de Cassiopée.https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/468Electron : l'icône de la notification de mise à jour ne marche pas lorsque l'...2020-09-24T12:16:48+02:00Mathias ChouetElectron : l'icône de la notification de mise à jour ne marche pas lorsque l'application est empaquetéeEn mode développement (lancer `electron .` une fois l'application compilée), la notification est affublée de l'icône attendue, lue dans `dist/assets/icons/electron/icon.png`. Mais lorsque l'application est empaquetée avec `electron-build...En mode développement (lancer `electron .` une fois l'application compilée), la notification est affublée de l'icône attendue, lue dans `dist/assets/icons/electron/icon.png`. Mais lorsque l'application est empaquetée avec `electron-builder`, une fois installée l'icône de la notification est remplacée par une icône par défaut.
Il semble que lorsque l'application est empaquetée dans un fichier `app.asar`, les chemins ne sont plus les mêmes… si on dépaquète avec `npx asar extract app.asar .`, le fichier `dist/assets/icons/electron/icon.png` est pourtant bien présent…
Essayé l'astuce suivante, sans succès : https://stackoverflow.com/questions/41823184/how-to-get-icon-path-image-in-electron-builder
Peut-être faut-il explicitement copier un fichier .png à côté du fichier app.asar, pour l'utiliser ensuite ? Mais comment ? Et quel sera son chemin ?
Note : avec Electron v8 ça marchait, le problème se produit à partir de la v9.https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/457Espèce personnalisée : grouper les critères de PAB2023-05-12T10:41:49+02:00Mathias ChouetEspèce personnalisée : grouper les critères de PABNécessite jalhyd#271Nécessite jalhyd#271Version 4.18.0https://gitlab.irstea.fr/cassiopee/nghyd/-/issues/453Prébarrage : améliorer l'algo de tri du graphique pour éviter les croisements2020-09-09T13:42:00+02:00Mathias ChouetPrébarrage : améliorer l'algo de tri du graphique pour éviter les croisementsMarche mieux avec l'algo gauche-droite, mais pas encore au point :
* charger le "gros exemple"
* ajouter une cloison B3-B5
* => croisementMarche mieux avec l'algo gauche-droite, mais pas encore au point :
* charger le "gros exemple"
* ajouter une cloison B3-B5
* => croisement