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

Fix remaining raft issues #2012

Merged
merged 15 commits into from
Jan 17, 2024
Merged

Conversation

casperlamboo
Copy link
Contributor

@casperlamboo casperlamboo commented Jan 11, 2024

Description

There were two issues remaining from PR #2006. The issues were the following

Issues with convex hull

image-20240110-103247
image-20240110-103453

Was caused by some unhandled edge cases in our convex hull implementation. Fixed here: c4846cb

Partially printed raft layers

image-20240110-123210
Fixed here: 37295d6

CURA-11395

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Yes

Test Configuration:

  • Operating System: MacOS 13.3

Checklist:

  • My code follows the style guidelines of this project as described in UltiMaker Meta
  • I have read the Contribution guide
  • I have commented my code, particularly in hard-to-understand areas
  • I have uploaded any files required to test this change

@casperlamboo casperlamboo force-pushed the CURA-11395-fix-convex-hull-raft-polygons branch from 72ab846 to 0265e74 Compare January 11, 2024 13:43
@saumyaj3 saumyaj3 merged commit 0b3c9d7 into main Jan 17, 2024
19 of 20 checks passed
@saumyaj3 saumyaj3 deleted the CURA-11395-fix-convex-hull-raft-polygons branch January 17, 2024 15:51
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.

4 participants