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

Marlin v1 ultimaker rc3 #1

Merged
merged 3 commits into from
May 9, 2012

Conversation

ErikDeBruijn
Copy link

Hey David,

Ik probeer mijn aanpassingen netjes in git te zetten zodat het wat transparanter is wat voor versie 'stock' is en makkelijker dan een zipfile op firmware.ultimaker.com.

Deze commits lijken me niet te ingrijpend. Als jij het de goede richting in vind gaat, graag mergen en mee upstream laten gaan. Ik heb gerebased en de commits zouden mooi moeten kunnen mergen.

groet,

Erik

…ed on MOTHERBOARD == 7 ? "Ultimaker" : "Mendel"
… You can now decide to allow/disallow large length of extrusions to be executed.

Turning off the protection might be beneficial to those who want to do fast and long reversals.
@ErikDeBruijn
Copy link
Author

Of course I should've written that in English ;)
Well, I'm trying to adopt the good behavior of working in git in a more transparent fashion. This first pull request (in a long time) is to get the ball rolling again.

@daid
Copy link
Owner

daid commented May 9, 2012

Changes look fine to me.

A few comments:

  • MOTHERBOARD 71 is also an Ultimaker.
  • I don't think "Store memory" is any clearer then "Store EEPROM". But I don't know anything better :P
  • M115 always reports 1 extruder. You can use http://gcc.gnu.org/onlinedocs/cpp/Stringification.html to get the EXTRUDERS define into a string.

daid added a commit that referenced this pull request May 9, 2012
@daid daid merged commit 830972c into daid:Marlin_v1 May 9, 2012
daid pushed a commit that referenced this pull request Dec 6, 2012
daid added a commit that referenced this pull request Mar 26, 2013
Fix step rate bug (especially for retraction)
daid added a commit that referenced this pull request Dec 11, 2013
Fix step rate bug (especially for retraction)
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