5KHz Tickle pulse too short.

[leswright1977]

First of all, this project is awesome! Beautiful work! Exactly what I was looking for!

When I Scoped the Tickle pulse, the width is too short for a Synrad 48. It should have a width of 1uS
Easy fix, in lib/LaserController/Synrad48Ctrl.h line 54 change this
const static uint16_t ticklePWM = 4;
To this:
const static uint16_t ticklePWM = 20;

I have also drawn a schematic and produced a PCB design for this project, and am happy to contribute it if you like. Also do you mind if I showcase this project on my YouTube Channel?

[Bx3mE]

Thanks for taking interest in the project! You are correct in "ticklePWM = 4" beeing wrong but thinking about it makes me wonder if 20 would be wrong too, in theory at least. Length of tickle should be 1/200 as defined by the synrad operators manual. in theory: in /lib/LaserController/Synrad48Ctrl.cpp: line 32 defines the resolution to 12 (4096 max pwm value) and in line 64 it is set. Given that requirement is 1/200 duty cycle the pwm should be 4096/200 or 20,48 Indeed you are correct but given it is a minimum requirement i think 21 is more appropriate. (not that it makes a big difference :P) Do you agree? PCB: Great! Please share or make a PR! And for YT, Please do!

…

On Mon, Jul 4, 2022 at 10:40 PM Les Wright ***@***.***> wrote: First of all, this project is awesome! Exactly what I was looking for! When I Scoped the Tickle pulse, the width is too short for a Synrad 48. It should have a width of 1uS Easy fix, in lib/LaserController/Synrad48Ctrl.h line 54 change this const static uint16_t ticklePWM = 4; To this: const static uint16_t ticklePWM = 20; I have also drawn a schematic and produced a PCB design for this project, and am happy to contribute it if you like. Also do you mind if I showcase this project on my YouTube Channel? — Reply to this email directly, view it on GitHub <#15>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEYS3MM5H67XX6G4Y7TKNBTVSND27ANCNFSM52UE6TMQ> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[leswright1977]

Awesome, sure I will go along with that, the Synrad won't mind +/- few ns and you know more about the timers in the Teensy than I do!
Great! I am glad I can contribute. The PCB files are hosted here: https://github.com/leswright1977/OPAL_PCB Feel free to clone and add to this project! Comments or suggestions welcome for the physical design.

Cheers!

[Bx3mE]

fixed and pr on your repo!

…

On Tue, Jul 5, 2022 at 3:21 PM Les Wright ***@***.***> wrote: Awesome, sure I will go along with that, the Synrad won't mind +/- few ns and you know more about the timers in the Teensy than I do! Great! I am glad I can contribute. The PCB files are hosted here: https://github.com/leswright1977/OPAL_PCB Feel free to clone and add to this project! Comments or suggestions welcome for the physical design. If you drop me a direct e-mail ***@***.***) I would be happy to send you a few unpopulated PCB's :-) Cheers! — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEYS3MMQKALSWP4VLYCN5JLVSQZGXANCNFSM52UE6TMQ> . You are receiving this because you commented.Message ID: ***@***.***>