Skip to content
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

Fsr #6

Open
wants to merge 34 commits into
base: fsr
Choose a base branch
from
Open

Fsr #6

wants to merge 34 commits into from

Conversation

eddieparker
Copy link

Hey Johann:

I've added two bits of functionality to Marlin/your FSR code. Feel free to take all, partial or none:

  • I added a define called FSR_ANALOG_READ.

If defined, the code remains unchanged (reads from thermistor). If undefined, it uses the Z_MIN to detect if the bed height has been hit or not.

For me this is essential as I'm using John Socha's board to send a digital signal (LOW) when hit, and I want to (one day) use my thermistor for the bed as intended.

  • I've started separating Configuration.h into MyConfiguration.h and Configuration.h.

I find it a bit cleaner - MyConfiguration.h is .gitignored, so people can have their own settings there, but not get stomped over by other people. MyConfiguration.h.example can show people how things can be done, and Configuration.h can be left as the #define logic processor it currently is.

Let me know if you have any questions, suggestions, or violent responses to this. :)

Cheers,

-e-

jcrocholl and others added 30 commits December 11, 2012 01:23
…figuration.h be more like "auto configuration". Makes merging easier when I can ignore peoples Configuration.h work.
Eddie Parker added 4 commits August 27, 2014 23:56
…not using it. I didn't realize, but the Neopixel libraries disable communications temporarily, which is *horrible* for 3D printers that rely on a host program communicating with this. I'll be moving this off to a Trinket next, and exposing a wire interface for my changes.
…efully remember not to push this specific change.
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.

4 participants