Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Article : OSM DATA 3D - Modélisation 2D des bâtiments (3/5) #1279

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Article : OSM DATA 3D - Modélisation 2D des bâtiments (3/5) #1279

wants to merge 11 commits into from
+389 −3

Conversation

TANK2003
Copy link
Contributor

Troisième article d'une série de 5 sur la nouvelle version d'OSM DATA (aka son printemps) en 3D. Dans ce troisième article, on commence la modélisation des données 2D issues des flux définient au dernier article. La modélisation architecturale des bâtiments issus d'OpenStreetMap, en commencant par les toitures, les facades feront l'objet du quatrième article

Merci d'avance pour votre relecture.

ℹ️ Message automatique à lire et remplir ⬇️

Liens importants

Ressources et check-list

Pour info, voici quelques extraits de notre guide de contribution :

Qualité

  • les news sont bien réparties dans les bonnes sections
  • les sections vides sont supprimées
  • vérifier le rendu de la syntaxe markdown (cf. linter)
  • chasse aux coquilles orthographiques et dyslexiques

Images

  • les images téléversées sur le CDN n'ont pas de caractère spécial dans leur nom de fichier (espace, accent, etc.) et n'excédent pas 1000px de largeur
  • les images sont hébergées sur des sites sécurisés (HTTPS)
  • chaque news a une vignette
  • les images (sauf les vignettes) ont l'attribut loading défini sur lazy (cf. MDN et guide material-mkdocs)
  • accessibilité : chaque image a un texte de remplacement (entre les crochets) et un titre lisible par les outils d'assistance. Bref, qui respecte la syntaxe générale

📢 Diffusion

Une fois l'article publié, il sera alors temps de le diffuser. Il sera automatiquement intégré au flux RSS et à la newsletter.

Mais la publication sur les réseaux sociaux est manuelle.

Nous avons des comptes officiels sur ces réseaux sociaux :

Merci d'indiquer en commentaire tes comptes à utiliser pour être cité/e dans les messages et de cocher ci-après la "stratégie" de diffusion qui te convient pour chaque réseau.

BlueSky

  • un/e membre de Geotribu publie, tu repartages avec ton compte
  • tu publies, on repartage
  • chacun/e publie de son côté
  • je souhaite que mon contenu ne soit pas diffusé sur ce réseau

LinkedIn

  • un/e membre de Geotribu publie, tu repartages avec ton compte
  • tu publies, on repartage
  • chacun/e publie de son côté
  • je souhaite que mon contenu ne soit pas diffusé sur ce réseau

Mastodon

  • un/e membre de Geotribu publie, tu repartages avec ton compte
  • tu publies, on repartage
  • chacun/e publie de son côté
  • je souhaite que mon contenu ne soit pas diffusé sur ce réseau

@github-actions github-actions bot added the articles Tutoriels, actualité, billet... label Mar 10, 2025
@Guts Guts requested review from a team and KazeNoOni and removed request for a team March 11, 2025 09:22
@Guts Guts changed the title init article osmdata 3 Article : OSM DATA 3D - Modélisation 2D des bâtiments (3/5) Mar 11, 2025
@Guts Guts requested a review from jeremieprudhomme March 11, 2025 09:23
@github-actions github-actions bot added rdp Revues de presse mkdocs Tout ce qui est lié au générateur de site : configuration, hooks, scripts... labels Mar 11, 2025
@Guts
Copy link
Member

Guts commented Mar 11, 2025

Allez zou, c'est parti pour la @geotribu/relecture !

ℹ️ Aux relecteur/ices

Merci d'utiliser autant que possible le mode Suggestion de la review : https://docs.github.com/fr/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request#starting-a-review

image

C'est vraiment IMPORTANT pour le confort de l'auteur/ice 🙏

@sguimarra si tu veux jeter un œil également, tu es le bienvenu !

@Guts
Copy link
Member

Guts commented Mar 11, 2025

@GlaDal reçois-tu les notifications GitHub ?

jeremieprudhomme
jeremieprudhomme reviewed Mar 11, 2025
View reviewed changes
Copy link
Contributor

@jeremieprudhomme jeremieprudhomme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voici mes commentaires suite à lecture. Pas grand chose cette fois-ci 😃

content/articles/2025/2025-03-10_osm-data-3D-03-modelisation-toiture.md Outdated

Dans le cas de toitures plus complexes et à des fins d'industrialisation de la solution, nous avons défini une formule de détermination générale de la hauteur pour chaque point de notre géométrie. Soit le point i, son altitude est définie de la manière suivante :

$Zi = *height* - *roof:height* + *roof:height* * (Dmax / Di)$
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le fait de mettre le nom des hauteurs entre astérisques rend confuse la lecture, surtout à la fin avec le * (Dmax/Di).
Est-ce que ça serait ok ok si tu enlèves les astérisques ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je suis bien d'accord, c'est fait 🙏

content/articles/2025/2025-03-10_osm-data-3D-03-modelisation-toiture.md
Comment on lines +366 to +370
<iframe height="500" style="width: 100%;" scrolling="no" title="Affectation de l'altitude à chaque sommet des triangles" src="https://codepen.io/TANK2003/embed/YPKELWd?default-tab=result" frameborder="no" loading="lazy" allowtransparency="true" allowfullscreen="true">
See the Pen <a href="https://codepen.io/TANK2003/pen/YPKELWd">
Affectation de l'altitude à chaque sommet des triangles</a> by Karl TAYOU (<a href="https://codepen.io/TANK2003">@TANK2003</a>)
on <a href="https://codepen.io">CodePen</a>.
</iframe>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est normal le 59m sur le sommet du fond du toit ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oui, c'est arrondi. Et ca "prouve" un peu l'utilisation de la formule.

TANK2003 and others added 5 commits March 11, 2025 21:02
@TANK2003 @jeremieprudhomme
Update content/articles/2025/2025-03-10_osm-data-3D-03-modelisation-t…
7a8dbf8 
…oiture.md

Co-authored-by: jeremieprudhomme <[email protected]>
Signed-off-by: Karl TAYOU <[email protected]>
@TANK2003 @jeremieprudhomme
Update content/articles/2025/2025-03-10_osm-data-3D-03-modelisation-t…
9dd8932 
…oiture.md

Co-authored-by: jeremieprudhomme <[email protected]>
Signed-off-by: Karl TAYOU <[email protected]>
@TANK2003 @jeremieprudhomme
Update content/articles/2025/2025-03-10_osm-data-3D-03-modelisation-t…
6624b12 
…oiture.md

Co-authored-by: jeremieprudhomme <[email protected]>
Signed-off-by: Karl TAYOU <[email protected]>
@TANK2003 @jeremieprudhomme
Update content/articles/2025/2025-03-10_osm-data-3D-03-modelisation-t…
7eac0e8 
…oiture.md

Co-authored-by: jeremieprudhomme <[email protected]>
Signed-off-by: Karl TAYOU <[email protected]>
@TANK2003
fix(osmdata3) : fix formula wording
f5bbd1d
@github-actions GitHub Actions
Copy link
Contributor

🎉 Le site de prévisualisation pour cette PR a été généré et est disponible sous ce lien.
Commit de référence : f5bbd1d.

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

@jeremieprudhomme jeremieprudhomme jeremieprudhomme left review comments

@KazeNoOni KazeNoOni Awaiting requested review from KazeNoOni KazeNoOni was automatically assigned from geotribu/relecture

Assignees

@TANK2003 TANK2003

Labels
articles Tutoriels, actualité, billet... mkdocs Tout ce qui est lié au générateur de site : configuration, hooks, scripts... rdp Revues de presse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TANK2003 @Guts @jeremieprudhomme