Unity-Technologies / Unity-Technologies/Unity.Mathematics
Rename quaternion.RotateX() and friends -> quaternion.XRotation()
Nessuno ha ancora preso questa issue.
- Lingua principale
- C#
- Stelle
- 1.4k
- Fork
- 159
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
There is inconsistency right now in the naming of quaternion factory methods, which can lead to confusion (see https://unity.slack.com/archives/CE7DZN2H1/p1546448661023000).
quaternion.RotateX() is named in a such a way that implies the method is going to rotate something, rather than produce a rotation quaternion (cf https://docs.google.com/document/d/1RaGSTzE3d3AN-l73J18nxKlPgIwLCzVdQuWdhYKywbA/edit). To bring them in line with LookRotation(), please consider renaming:
RotateX()->XRotation()RotateY()->YRotation()RotateZ()->ZRotation()
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia individuando quaternion.RotateX(), quaternion.RotateY() e quaternion.RotateZ() e verificandone i riferimenti nel repository Unity.Mathematics. Il lavoro è completato quando i metodi factory usano in modo coerente XRotation(), YRotation() e ZRotation() e i riferimenti interessati rimangono allineati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- csharp
- Ambito
- computer-graphics
- Tipo di issue
- Refactoring
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 38/100