-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
5KHz Tickle pulse too short. #15
Comments
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: ***@***.***>
|
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! Cheers! |
Bx3mE
added a commit
that referenced
this issue
Jul 5, 2022
Value of TicklePWM was updated to reflect the updated 5Khz Frequency -> Issue #15
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:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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?
The text was updated successfully, but these errors were encountered: