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

X axis offset on multiple model when changing color with AMS lite #4841

Closed
2 tasks done
jmilhau opened this issue Sep 19, 2024 · 38 comments
Closed
2 tasks done

X axis offset on multiple model when changing color with AMS lite #4841

jmilhau opened this issue Sep 19, 2024 · 38 comments
Assignees
Labels
bug Something isn't working

Comments

@jmilhau
Copy link

jmilhau commented Sep 19, 2024

Bambu Studio Version

1.9.7.50

Where is the application from?

Bambu Lab Official website

OS version

MacOS 15.0

Additional system information

Since today (since latest Studio update? v1.9.7.50) any print I make with several colors in the same layer gets a x-axis gap a several millimeters. Maintenance did not help.

image
image

Black was printed first, then purple then cyan then orange.

Printer

Bambu Lab A1 Combo

How to reproduce

Print a model with 2 colors on the same layer

Actual results

Offset of the first color on the print
image

Expected results

No offset
image

Project file & Debug log uploads

test v1.3mf.zip
Cannot add configuration folder (more than 25MB)

Checklist of files to include

  • Log file
  • Project file
@jmilhau jmilhau added the bug Something isn't working label Sep 19, 2024
@jmilhau
Copy link
Author

jmilhau commented Sep 19, 2024

Re installing 1.9.5.51 solves the issue

@BastR
Copy link

BastR commented Sep 19, 2024

Same issue on A1 with AMS lite on macOs, had to stop print.
IMG_3095

@MCKanpolat
Copy link

MCKanpolat commented Sep 19, 2024

I've just updated the latest version and same issue I have. Mac OS 15

Also reinstalling (clean install) the BambuStudio didn't solved the issue.

@BastR
Copy link

BastR commented Sep 19, 2024

I did a few tests with Windows and printing went off without a problem.

Edit: A user on the official facebook group reports that he also has the same problem with Windows.

@ziehmon
Copy link

ziehmon commented Sep 19, 2024

Are you using timelapse by any chance?

@BastR
Copy link

BastR commented Sep 19, 2024

In my case no, no timelapse on the A1

@jmilhau
Copy link
Author

jmilhau commented Sep 19, 2024

No timelapse either

@ziehmon
Copy link

ziehmon commented Sep 19, 2024

Alright, can you do another check just to make sure? Try to remove the timelapse_gcode in extruder settings. It's a wild guess but worth a try. The behavior was changed in the new release.

Printing the first layers will be probably enough.

@NicolasABSuarezB
Copy link

I have the same problem with mi A1 (Bambu Studio v1.9.7) on Windows
Solved with the reinstallation of Bambu Studio v1.9.5.51

Don't give importance to the texture of the first layer, I'm trying some things
1 9 5 51
1 9 7

@SaltWei
Copy link
Collaborator

SaltWei commented Sep 20, 2024

img_v3_02et_32c9eb96-c3d1-41f3-8a09-de7cd19e676g
Caused by this. Disable timelapse can workaround.

@SaltWei
Copy link
Collaborator

SaltWei commented Sep 20, 2024

Will revert this patch and hot fix this issue

@SaltWei SaltWei removed the Printer issues of Printer label Sep 20, 2024
@BastR
Copy link

BastR commented Sep 20, 2024

Bump.mp4

@SaltWei @ziehmon As you can see from the video, even with the timelapse_code removed, there is still a layer offset.

You can hear the bed collide with the movement of the printer head, so I suppose that's when the step is lost and there's a shift.

@XunZhangBambu
Copy link
Contributor

XunZhangBambu commented Sep 20, 2024

@BastR To solve your issue currently, you need to turn off timelpase when printing rather than remove the timelapse gcode
image

@jmilhau
Copy link
Author

jmilhau commented Sep 20, 2024

Timelapse was NEVER activated for me, so I think this does not solve the issue.

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

@XunZhangBambu when the timelapse gcode is deleted, get_last_pos_from_gcode() would not change the position. The user also had timelapse disabled.

Where do you think the bug is exactly?

It must be happening when AMS is used, because users without one don't face this issue it appears.

@BastR
Copy link

BastR commented Sep 20, 2024

@XunZhangBambu As @jmilhau says, the timelapse had never been activated when I made my prints, so I activated and deactivated the timelapse just in case, but I still have the shift.

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

@XunZhangBambu As @jmilhau says, the timelapse had never been activated when I made my prints, so I activated and deactivated the timelapse just in case, but I still have the shift.

You were testing with AMS, right? Could you test using only one color, so no changes in between?

@BastR
Copy link

BastR commented Sep 20, 2024

@XunZhangBambu As @jmilhau says, the timelapse had never been activated when I made my prints, so I activated and deactivated the timelapse just in case, but I still have the shift.

You were testing with AMS, right? Could you test using only one color, so no changes in between?

Yes all my prints was with AMS, with only one color no problem.

@jmilhau
Copy link
Author

jmilhau commented Sep 20, 2024

If I summarise:

  • What works: Printing with 1 color from v1.9.7.50 / Printing anything from v1.9.5.51
  • What does not work: Printing from v1.9.7.50 with 2 colors or more with AMS/AMS lite with or without Timelapse activated

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

@jmilhau @BastR gotcha. That's the hint. I authored the change that probably caused this and I didn't had an AMS to test. But your description is enough to find it.

Update: found the bug. Fix is comming @XunZhangBambu:

grafik

@XunZhangBambu
Copy link
Contributor

@ziehmon
image
The bug occurs in this change. Although the retract and lift are executed before inserting the timelapse G-code, the actual lift will be performed during next travel generated by code, at which point the position may already be at the extreme edge of the printing area. Performing another spiral lift at this point may exceed the machine's printing range, leading to lost steps

@BastR
Copy link

BastR commented Sep 20, 2024

IMG_3101.mp4

I deliberately shifted the X axis and you can see that during the last movement the head moves more towards 0.

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

@ziehmon image The bug occurs in this change. Although the retract and lift are executed before inserting the timelapse G-code, the actual lift will be performed during next travel generated by code, at which point the position may already be at the extreme edge of the printing area. Performing another spiral lift at this point may exceed the machine's printing range, leading to lost steps

I'm sorry, I don't yet agree with this analysis. There is plenty of room to execute the spiral Z hop. The problem is an inconsistency in the approached position when there is an AMS in use .

I am analyzing this, let's not jump to conclusions.

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

grafik

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

grafik

I found it. The position where the Z hop inserted via::
grafik

is different, when AMS is used. The head is at x=-48.2000 with AMS, and having a Z hop then indeed moves the head out of the bed's boundaries.

I think we just have to change the gcode for filament_change @XunZhangBambu. The head needs to be at x=0.00000 after it, then everything works fine. That's the difference between the AMS gcode and the one without :)

@XunZhangBambu
Copy link
Contributor

@ziehmon lets take the 3mf @jmilhau provided to analysize.We add a comment where we expect the lift happen and set zhop to a special number 0.366
image

then you will find the lift dooes't immediately happen

image
it happens in the line 1440 , and the nozzle is out of the bed
image

image

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

Yes you're right @XunZhangBambu. I agree and my analsys was only correct when no AMS is used.

The fix is simple: in filament_change_gcode, the printer shall move to x=0.00000 at the end, exactly like it does in timelapse_gcode. This then gives enough room to perform any upfollowing moves:

G1 X0 Y{first_layer_center_no_wipe_tower[1]} F18000 ; move to safe pos
M400

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

The screenshot you showed shows exactly that this is because of the position the head is in when filament_change_gcode is finished! :)

@XunZhangBambu
Copy link
Contributor

@ziehmon What you mentioned could be a solution. However, in order to ensure absolute safety, we need to revert your submit first. After fixes and thorough testing, it will be reintegrated. Thank you for your understanding.

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

Understood @XunZhangBambu. I'll create a new PR, including the change. Maybe you can start testing it soon. For a lot of people this fix massively improved print quality, but I understand that with AMS, this is a nasty bug.

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

I am sorry for this mess. I don't own an AMS and I was unable to test this scenario. The fix for stringing is very nice but it's uncool that it broke people's printers. Sorry guys. At least we can do it better with the next PR 🚬

@SaltWei
Copy link
Collaborator

SaltWei commented Sep 20, 2024

I am sorry for this mess. I don't own an AMS and I was unable to test this scenario. The fix for stringing is very nice but it's uncool that it broke people's printers. Sorry guys. At least we can do it better with the next PR 🚬

This is not your fault. We always welcome PR from community, and our tester should tested more carefully and enough.
Sorry everyone, we will hot fix this issue ASAP.

@jmilhau
Copy link
Author

jmilhau commented Sep 20, 2024

@ziehmon The fact that you are helping to solve the issue is way more important than the bug introduced. MANY THANKS for that. I would suggest Bambulab to send you an AMS for future testing ;)

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

@lanewei120 thanks for your warm words.

I created a new PR that handles this problem. There is a very extensive analysis inside: #4848. I also created a list of testcases we should print to make sure this is not going to happen again.

@XunZhangBambu would you be interested in reviewing and testing this, since you review the original PR?

@ziehmon
Copy link

ziehmon commented Sep 20, 2024

BTW, this was why the fix that caused this problem was introduced originally (printed without AMS....):

grafik

@SaltWei
Copy link
Collaborator

SaltWei commented Sep 20, 2024

BTW, this was why the fix that caused this problem was introduced originally (printed without AMS....):

I will ask our marketing colleague whether it's possible to send you one ams lite(not promise). Will let you known if be approved.
:)

@ziehmon
Copy link

ziehmon commented Sep 27, 2024

BTW, this was why the fix that caused this problem was introduced originally (printed without AMS....):

I will ask our marketing colleague whether it's possible to send you one ams lite(not promise). Will let you known if be approved.

:)

That would be absolutely amazing! Even a discount code would be a great gesture, but it's understandable if it doesn't get approved 🙂

@ttt888zzz
Copy link

The latest version has been fixed.
Please visit the website at https://github.com/bambulab/BambuStudio/releases and try again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

10 participants