-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
hblr review #20
Comments
Should also check current consumption on 5V_ISO, and consider going to a switchmode supply if high. |
+Add capacitor to PWM out for spindle control. Why below: Problem: The 40.0000 RPM spindle driver used in Hedy could not be controlled with the stock PWM speed control, it read the signal as always max. |
Updates added to latest version:
|
Super. Does this mean that the hrbl shield is now robust enough to be forked as a shopbot controller replacement? I remember we had noisy probe signal on Hedy PCB mill when the shopbot was running next to it. |
Yes, that is my opinion. If we in the future find that we need another capacitor or similar it's easy enough to add to the forked design. We could of course tweak and slim it indefinitely, to optimize component choices etc but I think it's good enough for forking! Regarding the noise on the probe: The stronger pull up on the arduino inputs should make it much more noise resistant and I have made the other suggestions that @jonnor suggested as well. |
Any more features we need before we order a set of these cards? |
In the Hedy version power in and polarity is not on the silk screen. Is this included in the gerber version for production? Is the Ardunio Nano name on the silkscreen? (not critical, but will help beginners assemble the card) Is the fabricatable Fabricatable machines logo still on? https://github.com/fellesverkstedet/fabricatable-machines/blob/master/wiki-media/fabricatable-machines-logo-single-stroke.dxf |
|
Adding 3 pin limit switch plugs on the board frees up another pin to be used as signal GND for alarm, making the plug wireing much simpler compared to splitting the GND yourself at the motor. And it is more flexible than having special motor PCBs that split the signals there. If we are doing motor cards then they should do more. |
Hot fix for the problem:
Details: The pinout of the loose L7805CV: Bend the legs and solder directly to the old pads like this And add the cooling fin to make sure it does not overheat. The card acted like a cooling fin previously, Alternatively, solder the new regulator into these vias and solder a wire to the unused middle pad. OBS! Before plugging it in, make double sure that the legs of the factory soldered regulator is no longer connected. |
Notes to next version:
|
Todo: |
Suggested changes, in order of priority
CC @Jaknil
The text was updated successfully, but these errors were encountered: