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

Improve post-processing scrirpt TimeLapse.py #19556

Closed
wants to merge 1 commit into from

Conversation

LemanRus
Copy link

@LemanRus LemanRus commented Aug 19, 2024

Description

This improvement is designed to enhance the Time Lapse script for better results:

Added a delay BEFORE taking the camera shot to reduce the impact of inertia and shake on the photo.
Modified the return-to-position code so that the nozzle avoids potential bumps, defects, etc. - first, the XY axes return, followed by the Z axis. This reduces the chance of the part shifting due to nozzle contact.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Test A: Timelapse with camera triggered by M118 command

Test Configuration:

  • Operating System: Windows
  • Printer: Ender 3 Neo

Checklist:

@github-actions github-actions bot added the PR: Community Contribution 👑 Community Contribution PR's label Aug 19, 2024
@GregValiant
Copy link
Collaborator

GregValiant commented Aug 21, 2024

I also did a re-write of TimeLapse. It's a bit more extensive. I'll attach it here. I've taken the liberty of adding your changes but I haven't updated the PR yet (#18923 ).

TimeLapse.zip

@GregValiant
Copy link
Collaborator

@LemanRus,
The new "Time Lapse" script has been merged and your request was included. Can we close this PR?

@LemanRus
Copy link
Author

Hi, @GregValiant
Yes, of course, you can
At least a lot of work has already been done on this little script)

@GregValiant GregValiant added Status: Duplicate Duplicate of another issue. PR: Post Processing ➕ Like adding beeps, more tunability or different Gcode pause at heights labels Oct 25, 2024
@GregValiant
Copy link
Collaborator

It's not as little as it was.
I'll go ahead and close this as it is completed with the merge of the other PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Community Contribution 👑 Community Contribution PR's PR: Post Processing ➕ Like adding beeps, more tunability or different Gcode pause at heights Status: Duplicate Duplicate of another issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants