RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
What changes to what files needs to be done if I install the Bondtech MK3 upgrade X axis carriage? Here it is:
https://www.bondtech.se/en/product/body-rebuild-set-update-mk2-3-to-mk3s/
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
I've ported the relevant changes into a new repository with sensible commit grouping. I'll be making that available later today. From that it'll be easy to create patches that can be applied to arbitrary versions of Prusa's firmware.
Any luck on this?
Yes, here: https://github.com/haarp/Prusa-Firmware/tree/09-steppers
This is based on Prusa's MK3 branch and I'll be occasionally rebasing it to stay up to date with it. Note that this is just the branch for changes to 0.9° steppers, ported from Kuo's. I don't want to pollute the point of a branch by merging everything and the kitchen sink into it. So no LA1.5, no extruder configs, no documentation.
You may see some style changes, I tried to keep things sensible (e.g. no wild mix of ifndef and ifdef). I also tried to separate commits sensibly. I'll try to maintain some order 🙂
I'm using 2450 for default homing speed, but you're free to fine-tune that.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Two thing, first an error and second an update about Bondtech upgrades:
First, the combined 0.9 github files do not compile for me. Did you remember that Prusa created their own board for Arduino? I get the following error while doing a compile test.
exit status 1
Error compiling for board PrusaResearch EINSY RAMBo.
Second, I found out the new Bondtech upgrade for the MK3 to MK33S upgrade needs a change. The mmu.cpp file needs to have this line in it, near the bottom after :static bool can_load()"
current_position[E_AXIS] -= 58;
Without it, the filament does not go down far enough to reach the nozzle before it starts to print. Or at least that is what they say, I cannot compile the firmware due to that error.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
I think you are addressing those comments to omnissiah
Now that he is maintaining his own 0.9 branch, I would prefer we split that off to a separate topic to avoid confusion with comments directed to my 0.9 branch.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
I think you are addressing those comments to omnissiah
Now that he is maintaining his own 0.9 branch, I would prefer we split that off to a separate topic to avoid confusion with comments directed to my 0.9 branch.
So he is handling the firmware located in your github? That is what I downloaded. I missed that handover, thanks!
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
No. I thought you had downloaded from his repository. It is confusing because there are now two separately maintained repositories for 0.9.
omnissiah is now maintaining his own repository. He is not maintaining mine.
It is now imperative to be explicit as to which repository one is using when making posts.
I thought you were using his because you mentioned the board change error which is already covered in my my branch's read me. You may be running into it for the first time because my branch has moved on to be 3.8.0 based.
So BMG for MK3S needs a new filament move distance definition?
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Maybe my install of Anduino broke, I will resintall. Your firmware worked recently, so it must be on my side. A PC reboot is in order too, always a good idea.
Yes, the E axis settings should be:
current_position[E_AXIS] += 60;
current_position[E_AXIS] -= 58;
It is down at the bottom in the section under "static bool can_load()" in the mmu.cpp file.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
As an aside, I found this out in the Bondtech Facebook group, they actually mentioned the guykou firmware as being the best to use, to make the needed changes as mentioned in your Github and make the -= change in the mmu.cpp file.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Added to my branch new define BONDTECH_PRUSA_UPGRADE_MK3S that activates the current_position[E_AXIS] -= 58;
Extruders defines in my branch are now...
//#define SKELESTRUDER
//#define BONDTECH_PRUSA_UPGRADE_MK3
//#define BONDTECH_PRUSA_UPGRADE_MK3S
//#define EXTRUDER_GEARRATIO_30
//#define EXTRUDER_GEARRATIO_3375 //(54:16 BNBSX
//#define EXTRUDER_GEARRATIO_35 // 3.5 like 56:16 Bunny and Bear Short Ears or Skelestruder.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
I agree with Kuo, let's not mix things up here. I created my repository mainly for myself, but several people asked me to publish it, hence why I posted it. I would recommend sticking with Kuo's repo, unless you know what you're getting yourself into.
(I can however say that it compiles fine on my end.)
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
I just got my printers parts in and swapped out the X and Y axis as well as built the BMG extruder. Downloaded the firmware file from https://github.com/guykuo/Prusa-Firmware/tree/0.9-Degree-Stepper-Support and followed the instructions. Getting a compile error and unsure how to resolve it.
I am able to compile the stock prusa firmware just fine.
The only things I had to change was removing the comment for the BMG extruder.
The first time I run to compile it gives me these errors mentioning issues with libraries in the temp folder
https://pastebin.com/U1neLJnU
If I try to immediately run it again it still fails to compile but doesn't have the exact same errors.
Not sure what is going on? I have tried to completely uninstall and remove Arduino and program/app data files, reinstall, re add the board. Can only successfully compile stock prusa firmware.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Hold on a bit. I think I know where the error crept in. I think it happened duirng the edit to add Bondtech MK3S.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
OK. Fixed. Somehow two of the source files were badly munged.
Should be good to go now.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
It compiled and uploaded just fine for me.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
My reading of the firmware's tmc2130_setup_chopper is that TMC2130_REG_IHOLD_IRUN register is being written using the same current values that are obtained from tmc2130_current_r array. That is populated from TMC2130_CURRENTS_R.
These are the same ones earlier found in Configuration.prusa.h
#define TMC2130_CURRENTS_H {16, 20, 35, 30} // default holding currents for all axes
#define TMC2130_CURRENTS_R {16, 20, 35, 30} // default running currents for all axesFirmware seems to verify these ARE used to set IRUN for stealtchop.
VERIFIED with printer and current probe. TMC2130_CURRENTS_H {16, 20, 35, 30} does affect stealthchop. However, how it works isn't entirely intuitive. Going too low on IRUN boosts coil current. If you set below 5, it goes so high that a few seconds of motion overheat the TMC2130
Above 22 and higher speed motion gets really weird sounding and distorted on scope.A value of 8-10 is okay in stealth chop, is lower than stock setting and still lets spreadcycle to move xy carriages well.
16 microsteps tried again.
In stealth, the max velocity gets throttled. That is probably why I could print in Stealth with 16 microsteps.
In spreadcycle, there isn't any speed throttling, but fast motion isn't well controlled - hence the layer shifts.I think we could run in 16 microsteps in spreadcycle but would need to more carefully go through firmware and limit feed rates.
I am going with
#define TMC2130_CURRENTS_H {10, 10, 35, 30} // default holding currents for all axes
#define TMC2130_CURRENTS_R {10, 10, 35, 30} // default running currents for all axesfor the time being. At least until the Moon's 0.9's get installed.
What are VFA's (Vertical Fine Artifacts)? They are fine vertical lines that appear on printed surfaces. They recur with each e-cycle of the stepper. Basically, the stock Prusa/LDO steppers don't move uniformly. They periodically accelerate/decelerate during each electrical step cycle. The uneven motion is synchronized with the step cycle and shows up as areas of greater and lesser filament deposition. Because x and y axis position remains in sync with motor step position, the VFA's remain in the same location. The VFA's remain in same horizontal position on x and y axis as printing advances up the object. We see these as fine vertical lines in print surfaces.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Guy-K2,
I am having a problem now with loading the filament. I have the Bondtech MK3S upgrade (new extruder body to use the extruder sensor, etc). The filament loads from the MMU2S just fine, enters the hot end, sets off the sensor, hits the gears, and pulls into the hot end as expected. Then it backs the filament up, literally pushing it OUT of the gears, then tries to pull it back in. Obviously, this will fail since it is no longer able to be grabbed by the gears.
Any idea what is causing that? Sliced the gcode using PrusaSlicer 2.0, freshly download and reinstalled today. Same thing happened with the 2.1 test versions.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Then it backs the filament up, literally pushing it OUT of the gears, then tries to pull it back in.
What version of firmware are you using? (When was it downloaded and compiled, even better zip an archive of your firmware source so I can definitively see the same as you are)
What options did you comment / uncomment?
I need to know the above to know which load distances are being used. I sounds like the constants for canload are wrong, but I don't know if your firmware matches my current.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
I downloaded your 0.9 firmware about 15 minutes before that post, maybe a 1/2 hour. I get notices when you merge and such And then I downloaded and make the below changes. I always look and see how long ago was that you made the Changes.
I uncommented the BMG MK3S line and that was it. I did verify that the X and Y were on commented, which they were come up and the language with proper, which it is.
I then tried Manually changing the E plus and minus settings in the MMU. CPP file by 10 higher and lower on each, one of the time, just to see if it would make any changes and it did not.
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Variant is 1_75mm_MK3S-EINSy10a-E3Dv6full?
RE: Stepper Motor Upgrades to Eliminate VFA's (Vertical Fine Artifacts)
Variant is 1_75mm_MK3S-EINSy10a-E3Dv6full?
Yes, sorry, forgot to add that. And then renamed as required.
I am not sure those E Axis settings do anything, I changed them by 10 to no effect. Ran the 415 code and rebooted after a well.