Skip to content

Conversation

@Testato
Copy link

@Testato Testato commented Jan 1, 2016

New Scheduler Example for multiple PWM

@Testato Testato changed the title Patch 1 MultiplePWM example Jan 1, 2016
@vbextreme
Copy link
Owner

@Testato penso che dovresti unire tutto in un unico file.
poi se vuoi puoi fare un esempio su piu file chiamato "examplemultisketch" dove ogni file ha il proprio setup ed il proprio loop, in modo da illustrare anche il metodo .start

@Testato
Copy link
Author

Testato commented Jan 6, 2016

Sono tab diverse non file diversi, concetto che non esiste in c/c++ ma a cui gli utenti arduino sono abituati.

Ho fatto così perché quando uscì la scheda TI che contiene un RTOS ricordo proprio la separazione su tre tab dei tre task, e la cosa colpì molto tutti.
Io farei al contrario, volendo usare il .start lo farei tutto in un unica Tab, proprio perché su Tab diverse a me fa pensare a processi paralleli mentre il .start, partendo "su comando" sta meglio in un unico file facendo pensare di più alla sequenzialità.
Che ne pensi ? Con gli esempi ufficiali si deve pensare più a come comunicare facilmente i concetti al lettore e non alla efficienza o comodità del codice.

@vbextreme
Copy link
Owner

Può andare ma sarebbe carino fare vedere il metodo .start, ovvero per ogni tab aggiungi .start(setup2) .startLoop(loop2).
Come se fossero piu sketch, cosi ogni tab si gestisce in toto il proprio led, dal setup al loop, cosa ne dici?

@Testato
Copy link
Author

Testato commented Jan 6, 2016

nessuno vieta di farne un terzo chiamato come dici tu multiplesketch.
in questo modo abbiamo il rpimo esempio con tutto in una tab, il secondo su piu' tab ma con il concetto del solo multiplo task, il terzo con il concetto di multiplo setup/loop.

@vbextreme
Copy link
Owner

riesci ad unire il tutto in un unico commit?

@Testato
Copy link
Author

Testato commented Jan 7, 2016

ok committato il nuovo esempio, ma.....

non funziona, se togli il solo setup3 inizia a funzionare.

bug o limite RAM ?

@Testato Testato changed the title MultiplePWM example New examples Jan 7, 2016
@vbextreme
Copy link
Owner

sembra un piccolo bug, riprova adesso ho fatto un nuovo pull

@Testato
Copy link
Author

Testato commented Jan 7, 2016

ora va, pero' viene fuori questo warning, c'e' sempre stato ?

WARNING: library Scheduler claims to run on [sam architecture(s) and may be incompatible with your current board which runs on samd] architecture(s).

vbextreme added a commit that referenced this pull request Jan 13, 2016
@vbextreme vbextreme merged commit aee3187 into vbextreme:master Jan 13, 2016
@Testato Testato deleted the patch-1 branch January 13, 2016 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants