Yarn: Forme Voxel -> Forme

Créé le 8 mai 2019  ·  5Commentaires  ·  Source: FabricMC/yarn

Hot Take Of The Week™️

Il n'y a rien d'intrinsèquement "voxel" dans l'implémentation, cela représente juste une collection de points qui forment une sorte de forme. Le seul point où les voxels sont introduits en tant que concept se trouve dans les méthodes tierces fournies dans la classe Block qui divisent les points donnés par 16,0 (le concept normalisé de voxel par cube 16x16x16) avant de construire une forme. Mais en dehors de cela, dans les classes et types primaires, il n'y a pas d'application de ce concept et ce sont purement des points représentant une forme qui peut être de n'importe quelle dimension ou résolution. Je reconnais que Mojang les appelle VoxelShape dans String et que Yarn peut avoir tendance à respecter les décisions de dénomination de Mojang, mais je pense que c'est l'un des nombreux cas où le nom pourrait facilement être simplifié car le préfixe peut être considéré comme redondant ou conceptuellement incorrect. Une autre réflexion à ce sujet est qu'un cas dans Advancement où le toString de Mojang le nomme en fait SimpleAdvancement a été reconnu comme incorrect et que Mojang appelle en fait la classe Advancement , et cela pourrait très bien être le cas pour VoxelShape aussi. En fait, Mojang ne l'appelle VoxelShape que 2 fois dans la base de code : le toString de VoxelShape et un message IllegalArgumentException dans ArrayVoxelShape. Pendant ce temps, également dans VoxelShape, il y a un message UnsupportedOperationException le décrivant simplement comme une forme ( "No bounds for empty shape." ). Je n'ai pas grand-chose de plus à mettre ici, donc je vais simplement conclure en rappelant à tout le monde qu'il s'agit d'une "prise chaude" et que je ne pousse pas pour ce changement, c'est simplement une suggestion générique.

Commentaire le plus utile

Pendant FAP Fry l'a appelé plusieurs fois VoxelShape. Je pense qu'il est assez sûr de supposer. Ferons-nous ?

Tous les 5 commentaires

Voxel signifie que les points de la grille sont alignés, je crois. Si les points ne peuvent pas être sur un point de réseau voxel, je soutiens définitivement la suppression du préfixe Voxel .

Voici donc le contre-problème avec la prise à chaud : les Voxels représentent en effet autre chose dans la programmation informatique générale et la conception de jeux, comme décrit par cet article wikipedia ici .

Mais dans le contexte de Minecraft, Voxel a été une terminologie courante pour le concept de pixels dans un blockspace, cependant, ces formes peuvent être utilisées n'importe où librement sans lien explicite avec les coordonnées de bloc. Cela dépend vraiment si vous considérez que Shape est suffisamment clair pour impliquer le concept littéral de voxels comme on le voit dans d'autres environnements de programmation. Nous ne voyons boîtes englobantes et vecteurs et savent qu'ils représentent des points qui pourraient être liés à une position mondiale en cas de besoin, et je me sens le même comportement pourrait être déduit du nom simplifié Shape aussi.

Il n'y a rien d'intrinsèquement "voxel" dans la mise en œuvre,

VoxelShape s sont faits de voxels. La forme du voxel est un tableau booléen en 3 dimensions (voxels) avec une taille pour chaque ligne, colonne et couche.

Si les points ne peuvent pas être sur un point de réseau voxel, je soutiens définitivement la suppression du préfixe Voxel.

Ils doivent être des points sur un réseau cuboïde.

Pendant FAP Fry l'a appelé plusieurs fois VoxelShape. Je pense qu'il est assez sûr de supposer. Ferons-nous ?

Ceci a été créé en sachant que Mojang l'appelle VoxelShape . Il a été conçu comme une simplification du concept de type. En moyenne, les votes penchent actuellement vers une baisse de ce changement, je vais donc le clôturer sur cette base.

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

asiekierka picture asiekierka  ·  3Commentaires

Bixilon picture Bixilon  ·  5Commentaires

altrisi picture altrisi  ·  4Commentaires

Boundarybreaker picture Boundarybreaker  ·  3Commentaires

quat1024 picture quat1024  ·  6Commentaires