IGNF / trainminator

Outil de labellisation développé en interne à l'IGN/DSI/(SDM et SIMV)
GNU General Public License v3.0
4 stars 0 forks source link

Mettre le plugin sur l'espace des dépôts QGis #48

Closed ngonthier closed 2 months ago

ngonthier commented 5 months ago

Non prioritaire

ngonthier commented 4 months ago

Licence GPL v3

ngonthier commented 3 months ago

Cela est maintenant possible suite à #60

CelestinHuet commented 3 months ago

Le plugin est trop gros... A priori, on peut supprimer le répertoire docs ? Il date d'il y a deux ans et il contient un énorme chantier qui ne doit sans doute plus être compatible avec le code actuel

ngonthier commented 3 months ago

@samysung et @LMartelet vous en pensez quoi ?

Pour ma part, je pense que l'on peut supprimer le répertoire docs pour la soumission plugin QGIS et ajouter au repo (non soumis sur QGIS une version zippé du chantier de test) ainsi qu'une mention dans le README sur comment télécharger ce chantier test. Je pense ça peut être utile quand même de l'avoir pour tester l'outil.

En outre, vous pouvez déplacer les deux documents Documentation_utilisateur.pdf et Documentation_developpeur.pdf si besoin (ainsi que les liens correspondants dans le README).

CelestinHuet commented 3 months ago

Il y a déjà un exemple de projet beaucoup plus léger dans le répertoire qgisproject_example. La seule chose qu'il y manque, ce sont les patches gris pour indiquer les zones déjà réalisées, mais cela ne me semble pas utile pour un chantier de démo

ngonthier commented 3 months ago

Il s'agit d'un projet mono date ? Est ce que l'on aurait un petit chantier bidates à rajouter pour avoir les deux à disposition ?

LMartelet commented 3 months ago

Ca doit se trouver dans tous les chantiers RPG ça

CelestinHuet commented 3 months ago

C'est un projet bidate

LMartelet commented 3 months ago

@samysung et @LMartelet vous en pensez quoi ?

Pour ma part, je pense que l'on peut supprimer le répertoire docs pour la soumission plugin QGIS et ajouter au repo (non soumis sur QGIS une version zippé du chantier de test) ainsi qu'une mention dans le README sur comment télécharger ce chantier test. Je pense ça peut être utile quand même de l'avoir pour tester l'outil.

En outre, vous pouvez déplacer les deux documents Documentation_utilisateur.pdf et Documentation_developpeur.pdf si besoin (ainsi que les liens correspondants dans le README).

Ok pour supprimer le gros chantier dans doc. Et pareil pour les docs users et devs

samysung commented 3 months ago

Oui, on peut simplement supprimer le projet Qgisà l'intérieur de Docs, et garder docs au cas où plus tard le code est maintenu ?

CelestinHuet commented 3 months ago

Je crée la pull request alors

samysung commented 3 months ago

en revnache, @CelestinHuet , ne serait-il pas préférable de mettre les champs qgisMinimumVersion et qgisMaximumVersion au plus sure? J'ai quelques doutes par exemple que trianminator fonctionne sur toutes les versions au dessus de 3 (à vrai dire j'ai vu des crashs avec une version en 3.16 je crois).

CelestinHuet commented 3 months ago

Je le modifie aussi

LMartelet commented 3 months ago

en revnache, @CelestinHuet , ne serait-il pas préférable de mettre les champs qgisMinimumVersion et qgisMaximumVersion au plus sure? J'ai quelques doutes par exemple que trianminator fonctionne sur toutes les versions au dessus de 3 (à vrai dire j'ai vu des crashs avec une version en 3.16 je crois).

Je t'assure qu'en 3.16 la version qu'on a ne marche pas. Perso j'ai surtout fait en 3.34 et je crois qu'en 3.28 ça ne marche toujours pas (je fais un check rapide)

CelestinHuet commented 3 months ago

Je mets 3.34 en version minimale, comme ça on ne s'embête pas

LMartelet commented 3 months ago

En fait 3.28 ça à l'air de passer. Mais bon 3.34 on est certain que ça fonctionne bien 😄

ngonthier commented 3 months ago

De mon côté cela fonctionne en 3.32

samysung commented 3 months ago

3.34 est une LTR (support long terme), c'est peut-être mieux de se baser là-dessus en effet

samysung commented 3 months ago

@CelestinHuet , bon bah tu nous dit quand on peut tester le téléchargement de trainminator depuis le gestionaire de plugin Qgis :)!

CelestinHuet commented 3 months ago

bah pour l'instant : Web server is returning an unknown error

Le serveur qgis a l'air d'être planté...

samysung commented 3 months ago

Ils ont pas payé leurs mensualités AWS peut-être

CelestinHuet commented 3 months ago

De ce que je comprends, si on essaye d'ajouter un plugin trop gros, alors toutes les tentatives suivantes renvoient un "Web server is returning an unknown error" jusqu'à ce que l'on se reconnecte. J'ai supprimé le répertoire asset et le chantier qui restait. La limite devait sans doute être à 10 Mo. Si certains ont besoin de ces deux répertoires, ils iront les chercher sur le Github. Le plugin est en cours de validation

samysung commented 3 months ago

l'extension trainminator à partir du dépôt sur Qgis 3.34 semble fonctionner chez moi.