nannou-org/nannou

Fix orientation of teapot in `wgpu_instancing.rs` after update to 0.5

Open

#596 ouverte le 10 mai 2020

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Rust (254 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (4 662 stars)
Métriques de merge PR
 (Merge moyen 24j 15h) (26 PRs mergées en 30 j)

Description

I just realised that I forgot to correct the orientation of the teapot in this example to account for wgpu's switch to having the y axis increase upwards (rather than downwards). Previously the teapot examples all used a view matrix with an inverted axis in order to account for the downwards y axis, as the original teapot vertices were intended for an upward y axis. This has been correct in the wgpu_teapot* examples in #594, but I forgot to correct it in wgpu_instancing.rs. This is a reminder to myself (or anyone) to fix it.

Guide contributeur

Direction de recherche
Mettez à jour la matrice de vue ou la transformation dans wgpu instancing.rs pour tenir compte de l'axe y vers le haut dans wgpu 0.5.
Stack technique
rust
Domaine
computer graphics
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80