← Retour aux issues

Blocus de la page /lieux

publié le , mis à jour
Avatar Codeberg de n4n5n4n5

La page lieux a été bloqué avec plusieurs commit

https://codeberg.org/cartes/web/commit/f62b4179335f233c9459919932f76043d84e535b https://codeberg.org/cartes/web/commit/7f7e02487d25ece09d7b8465c6f582bfae0fdb45 https://codeberg.org/cartes/web/commit/0ca3cc32e2aadf7f7a4a768d6827009eb3760359 https://codeberg.org/cartes/web/commit/5ec0234595098e696c33b3d266b55dae14488835

Car ça utilisait trop l'API overpass

Je propose de rétablir la page /lieux, mais pas les page qui utilisent overpass

https://codeberg.org/cartes/web/pulls/2895

Pour réactiver l'annuaire des categories sur un lieu, il faut enlever le blocus mis avec

category = null
  1. Avatar Codeberg de n4n5n4n5
  2. Avatar Codeberg de n4n5n4n5

    Remise du blocage complet avec pour side effect de bloquer /lieux

    https://codeberg.org/cartes/web/commit/5857b263e140bbe8de552ef2fedb18e128130b99

  3. Avatar Codeberg de n4n5n4n5

    Moi j'aime bien la page /lieux, et normalement elle n'utilise pas overpass (voir premier commentaire)

    Alors j'ai push https://codeberg.org/cartes/web/pulls/3013

    @etienneJr @laem

  4. Avatar Codeberg de etienneJretienneJr

    2 remarques sans lien :

    • ça m'intrigue : quel intérêt tu lui trouves si les requêtes overpass sont désactivées ?
    • je pense que les requêtes overpass des pages /lieux sont négligeables par rapport à celles des pages /lieu et /?allez, donc on pourrait les rétablir dès maintenant, mais bon, vaut ptêt mieux attendre que Maël ait bossé le blocage des bots.
  5. Avatar Codeberg de n4n5n4n5

    J'aimais bien la page pour voir les villes et les régions

    aussi le png de la ville qui est généré est sympa.

    Mais j'ai compris que l'image de la ville on peut la faire avec un binaire depuis maplibre-native.

    dcp peut etre que je créerai ma propre page /lieux dans un autre projet

    EDIT: https://n4n5.dev/can-be-useful/js-france-cities/

  6. Avatar Codeberg de laemlaem

    Mais j'ai compris que l'image de la ville on peut la faire avec un binaire depuis maplibre-native.

    C'est ce qu'on fait avec noter API placeImages :)

  7. Avatar Codeberg de n4n5n4n5

    @laem wrote in https://codeberg.org/cartes/web/issues/2896#issuecomment-22796173:

    Mais j'ai compris que l'image de la ville on peut la faire avec un binaire depuis maplibre-native.

    C'est ce qu'on fait avec noter API placeImages :)

    Non justement il me semble, ça utilise https://github.com/consbio/mbgl-renderer

    https://codeberg.org/cartes/serveur/src/commit/5c0aed661d0666ba8f3f2bfd88f94eae9da86153/placeMap.js#L14-L31

    Qui est du javascript autour du binaire maplibre-native

  8. Avatar Codeberg de n4n5n4n5

    J'ai mis un binaire ici pour info

    https://github.com/Its-Just-Nans/testing/actions/runs/34855818466

    si jamais du veux faire des tests

    ./cartes-placemap --lat 16.2712 --lng -61.5053

    Entre 1 et 3 seconds (selon le réseau) sur mon ordi

  9. Avatar Codeberg de laemlaem

    Mais je pige pas ce que tu veux dire. mbgl-render utilise maplibre-gl-native

  10. Avatar Codeberg de n4n5n4n5

    @laem wrote in https://codeberg.org/cartes/web/issues/2896#issuecomment-22815088:

    Mais je pige pas ce que tu veux dire. mbgl-render utilise maplibre-gl-native

    server express (JS) -> mbgl-render (JS) -> @maplibre/maplibre-gl-native (JS) -> maplibre-gl-native (CPP)

    Le binaire n'a pas de JS, ça pourrait donc améliorer les perfs, ça serait quasiment

    server express (JS) -> maplibre-gl-native (CPP)

  11. Avatar Codeberg de laemlaem

    Ah d'acc ! Je doute que les deux étapes du milieu soient limitantes, mais si t'arrives en effet à les éviter sans trop de boulot en plus, n'hésite pas !

  12. Avatar Codeberg de n4n5n4n5

    @laem wrote in https://codeberg.org/cartes/web/issues/2896#issuecomment-22838005:

    Ah d'acc ! Je doute que les deux étapes du milieu soient limitantes, mais si t'arrives en effet à les éviter sans trop de boulot en plus, n'hésite pas !

    Oui, c'est ce que je propose, faire un petit benchmark sur le serveur cartes.app entre

    Au niveau des arguments, c'est quasiment la même chose (a part -x, -y qui deviennent --lng et --lat

    cartes-placemap --style http://cartes.app/api/styles?key=base --output ${hash} -z ${zoom} --lng ${lon} --lat ${lat} -b ${bearing} -p ${pitch}

/.bun/bin/mbgl-render --style http://cartes.app/api/styles?key=base --output ${hash} -z ${zoom} -x ${lon} -y ${lat} -b ${bearing} -p ${pitch}

    Et si ça c'est concluant, on pourrait directement utiliser le binaire.

✏️ Participer à la discussion