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

2.3.57: Double Travel Between Perimeters #1560

Closed
CCS86 opened this issue Sep 13, 2021 · 9 comments
Closed

2.3.57: Double Travel Between Perimeters #1560

CCS86 opened this issue Sep 13, 2021 · 9 comments
Labels
fix is live in the last release Please download /build the last release and try to reproduce. problem

Comments

@CCS86
Copy link

CCS86 commented Sep 13, 2021

I wanted to double check if this was expected behavior. It seems like splitting the move like this could slow it down with jerk/accel limits:

image

Objective Lens Cap.3mf.txt

@supermerill
Copy link
Owner

I don't understand.

@CCS86
Copy link
Author

CCS86 commented Sep 27, 2021

Instead of a single travel move, from the end of one perimeter to the start of the next, it is splitting it up into 2 travel moves.

@supermerill
Copy link
Owner

that will add a useless line in the gcode, no?

@CCS86
Copy link
Author

CCS86 commented Sep 27, 2021

I don't follow.

I'm saying the the current build, there are 2 travel moves, instead of one. To me it seems like a useless line of code, that could cause motion control to execute it slower than a single move. I wanted to check if you had a specific reason for doing this.

@gpread
Copy link

gpread commented Sep 27, 2021

My intuition makes me wonder if this is related to #1550 ?

@supermerill
Copy link
Owner

I'm saying the the current build, there are 2 travel moves, instead of one.

weird, I saw only one.

@qwewer0
Copy link

qwewer0 commented Sep 27, 2021

I'm saying the the current build, there are 2 travel moves, instead of one.

weird, I saw only one.

Objective Lens Cap.3mf.txt

@supermerill Try at Z=7mm, 40298-40300.

@supermerill
Copy link
Owner

supermerill commented Sep 27, 2021

image
it's at 7.2
something is wrong.
what version?

@qwewer0
Copy link

qwewer0 commented Sep 28, 2021

it's at 7.2
something is wrong.
what version?

I tried it with 2.3.57, but here is a video of it:

bandicam.2021-09-28.06-13-55-431.mp4

@supermerill supermerill added the fixed for the next version That means that you should be able to test it in the latest nightly build label Nov 6, 2021
@supermerill supermerill added fix is live in the last release Please download /build the last release and try to reproduce. and removed fixed for the next version That means that you should be able to test it in the latest nightly build labels Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix is live in the last release Please download /build the last release and try to reproduce. problem
Projects
None yet
Development

No branches or pull requests

4 participants