micro-ROS / micro-ROS/micro_ros_azure_rtos_app
Incorrect definition of TX_TIMER_NANOSECOND_PER_TICK?
Nessuno ha ancora preso questa issue.
- Lingua principale
- C
- Stelle
- 13
- Fork
- 7
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
Hi,
I had a lot of troubles with my Micro ROS transportation reusing your microros_time.c file. It is possible that the definition of TX_TIMER_NANOSECOND_PER_TICK is incorrect: https://github.com/micro-ROS/micro_ros_azure_rtos_app/blob/49cd0df65aa717001f92beb5ac324fa5de629af4/app/microros_time.c#L5
Something like below would make more sense:
#define TX_TIMER_NANOSECOND_PER_TICK ((ULONG)(1000000000UL / TX_TIMER_TICKS_PER_SECOND))
Guida per i contributori
Apri la guida per i contributori
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
Esamina app/microros_time.c alla riga 5 e le definizioni dei timer circostanti. Confronta il calcolo attuale di TX_TIMER_NANOSECOND_PER_TICK con TX_TIMER_TICKS_PER_SECOND e con la formula proposta; il lavoro è completato quando viene confermata la conversione corretta e la definizione viene aggiornata se necessario.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- c
- Ambito
- embedded-iot
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100