Review or remove facility to measure gaze steadiness with a target object.

Chiusa
#85 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
52/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
csharp
Ambito
game-dev

Direzione di ricerca

The issue mentions the gaze steadiness code and its GetTargetCentre entry point; start by locating those references and checking whether the facility is used elsewhere. Review the target-object behavior and define completion as removing the unused facility, or retaining it only with a clear reason supported by the usage check.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug

We don't use this code, could remove it.

This isn't quite right if there is no target object set. GetTargetCentre will return (0,0) so the text will say "look here -> 0, 0" whereas it doesn't care where you look as long as your gaze is steady. However I don't think it's worth fixing as we've never used (and probably never will use) the gaze steadiness code without a target object.

Originally posted by @thompson318 in #84

Lingua principale
C#
Stelle
1
Fork
1
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di UCL/AstroBalance

Tutte le issue di UCL/AstroBalance

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.