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

Support Critical Regions Only No longer performs as expected #2195

Closed
1 of 2 tasks
anthonyp-cns opened this issue Sep 21, 2023 · 2 comments
Closed
1 of 2 tasks

Support Critical Regions Only No longer performs as expected #2195

anthonyp-cns opened this issue Sep 21, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@anthonyp-cns
Copy link

anthonyp-cns commented Sep 21, 2023

OrcaSlicer Version

1.7.0

OS version

Windows 10 22H2

Additional system information

CPU: i7-1195G7
Memory: 32Gb
GPU: RTX3050

Printer

Bambu Lab X1C

How to reproduce

Enable support
Enable support critical regions only

Actual results

The critical regions only option appears to have no effect.

Version 1.6.6 performed as expected supporting only critical regions, while 1.7.0 adds support everywhere

Expected results

Support of only critical regions instead of full model

Project file & Debug log uploads

Honeycomb Cup - orca 1.7.0 critical regions support issue.zip

Checklist of files to include

  • Log file
  • Project file
@anthonyp-cns anthonyp-cns added the bug Something isn't working label Sep 21, 2023
Noisyfox added a commit to Noisyfox/OrcaSlicer that referenced this issue Sep 22, 2023
@Noisyfox
Copy link
Collaborator

The default style tree is broken in 1.7, switch to a non-default style (i.e., TreeHybrid) instead.

SoftFever pushed a commit that referenced this issue Sep 24, 2023
* Organic supports: Added check for variable layer height, with which
Organic supports are not compatible.
Fixes prusa3d/PrusaSlicer#9528 and similar.
Check the object max Z against build volume Z in Print::validate().

Cherry-picked from prusa3d/PrusaSlicer@5b94971

* Fix crash with default tree support

* Show "support_critical_regions_only" only when using auto normal tree supports (#2195)

* Fix organic tree check

---------

Co-authored-by: Vojtech Bubnik <bubnikv@gmail.com>
@anthonyp-cns
Copy link
Author

Wow you are on top of it! Thanks for the quick response and submitting the code to fix this.

I am going to close this bug.

Thanks 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

2 participants