Skip to content

fix(examples): now water and glaciers are distinguished - #174

Draft
naulan-chrzaszcz wants to merge 1 commit into
mainfrom
naulan/paramsOfMeije
Draft

fix(examples): now water and glaciers are distinguished#174
naulan-chrzaszcz wants to merge 1 commit into
mainfrom
naulan/paramsOfMeije

Conversation

@naulan-chrzaszcz

@naulan-chrzaszcz naulan-chrzaszcz commented May 4, 2026

Copy link
Copy Markdown
Member

Changes

Mise à jour de full.yaml pour remplacer fetchWater par fetchHydrographic et ajout d'une nouvelle heightmaps snowfields afin de différencier les cours d'eau des glaciers.

Ajout de meije.yaml (montagne de La Meije) dans les exemples afin de mettre en valeurs les ombres de la minimap et l'ajout des glaciers.

Self-checks

  • The code has unit tests associated
  • The code has Javadoc Comments associated
  • Complex / Unexpected code is explained / justified with a small comment
  • Relevant documentation inside the /docs folder has been updated
  • All examples in examples/ work the same (or have been adapted if subject to changes in this PR)
  • Git history is clean (each commit accomplish a single task and describe it accordingly)
  • The texts have been proofread (documentation, error messages, logs, comments...)

@indyteo indyteo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Dans l'ensemble ça va aller, juste un choix de voxel à trancher et une simplification du traitement possible qui mène au même rendu

format: minecraft
references:
- &voxel-stone minecraft:stone
- &voxel-snowfields minecraft:snow_block

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tu utilises ça pour représenter les glaciers, ça me semble un peu étrange de choisir de la neige (et d'appeler ça "snowfields").
Certes les névés sont des amas de neige, mais qui a durci, et vu qu'on n'a pas de moyen de différencier les deux, je dirais que la neige dure s'apparente plus facilement à de la glace que le contraire... Quelque chose comme de la Packed Ice devrait faire l'affaire.
D'autant plus que pour les générations en montagne, on pourra activer le mode neige pour ajouter une couche de neige partout, si on veut (d'où le fait de se concentrer ici à mon sens sur les glaciers)

Comment on lines +207 to +216
populateSnowfieldsPresence:
after: fetchHydrographic
type: copyHeightmap
models:
type: hydrographic
filter:
metadata: nature
equals: "Glacier, névé"
from: 1
to: snowfields

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pourquoi passer par une heightmap comme pour l'eau alors qu'on ne fait pas d'applanissement ? On pourrait faire beaucoup plus simple en faisant comme avec les squares ou les étendues d'herbe, c'est-à-dire laisser le sol se générer, et l'écraser avec un nouveau sol (en glace) à la place

Comment on lines +263 to +276
renderSnowfields:
after:
- renderGround
type: renderHeightmap
minimum:
sum:
- ground
- 1
- product: [ -5, snowfields ]
maximum: ground
place:
structure:
- put: *voxel-snowfields
at: [ 0, 0, 0 ]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

D'autant plus qu'ici, au lieu d'un render heightmap, avec un min/max compliqué, tu aurais une simple structure de 1x1x5 de glace posée à la hauteur du sol, avec le render surfaces sur les bons models

@naulan-chrzaszcz naulan-chrzaszcz self-assigned this Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants