Proper fix for preventing shells to propagate too much
This commit is contained in:
parent
6bd4803612
commit
25af3eb35f
@ -603,7 +603,14 @@ sub discover_horizontal_shells {
|
||||
|
||||
# find intersection between neighbor and current layer's surfaces
|
||||
# intersections have contours and holes
|
||||
my $new_internal_solid = intersection_ex(
|
||||
# we update $solid so that we limit the next neighbor layer to the areas that were
|
||||
# found on this one - in other words, solid shells on one layer (for a given external surface)
|
||||
# are always a subset of the shells found on the previous shell layer
|
||||
# this approach allows for DWIM in hollow sloping vases, where we want bottom
|
||||
# shells to be generated in the base but not in the walls (where there are many
|
||||
# narrow bottom surfaces): reassigning $solid will consider the 'shadow' of the
|
||||
# upper perimeter as an obstacle and shell will not be propagated to more upper layers
|
||||
my $new_internal_solid = $solid = intersection_ex(
|
||||
[ map @$_, @$solid ],
|
||||
[ map $_->p, grep { $_->surface_type == S_TYPE_INTERNAL || $_->surface_type == S_TYPE_INTERNALSOLID } @neighbor_fill_surfaces ],
|
||||
undef, 1,
|
||||
|
15
t/shells.t
15
t/shells.t
@ -70,21 +70,26 @@ use Slic3r::Test;
|
||||
|
||||
{
|
||||
my $config = Slic3r::Config->new_from_defaults;
|
||||
$config->set('perimeters', 0);
|
||||
# we need to check against one perimeter because this test is calibrated
|
||||
# (shape, extrusion_width) so that perimeters cover the bottom surfaces of
|
||||
# their lower layer - the test checks that shells are not generated on the
|
||||
# above layers (thus 'across' the shadow perimeter)
|
||||
$config->set('perimeters', 1);
|
||||
$config->set('fill_density', 0);
|
||||
$config->set('cooling', 0); # prevent speed alteration
|
||||
$config->set('first_layer_speed', '100%'); # prevent speed alteration
|
||||
$config->set('extrusion_width', 0.2);
|
||||
$config->set('extrusion_width', 0.6);
|
||||
$config->set('bottom_solid_layers', 3);
|
||||
$config->set('top_solid_layers', 0);
|
||||
|
||||
my $print = Slic3r::Test::init_print('V', scale_xyz => [2,1,1], config => $config);
|
||||
my $print = Slic3r::Test::init_print('V', config => $config);
|
||||
my %layers = (); # Z => 1
|
||||
Slic3r::GCode::Reader->new(gcode => Slic3r::Test::gcode($print))->parse(sub {
|
||||
my ($self, $cmd, $args, $info) = @_;
|
||||
$layers{$self->Z} = 1 if $info->{extruding};
|
||||
$layers{$self->Z} = 1
|
||||
if $info->{extruding} && ($args->{F} // $self->F) == $config->solid_infill_speed*60;
|
||||
});
|
||||
is scalar(keys %layers), 3,
|
||||
is scalar(keys %layers), $config->bottom_solid_layers,
|
||||
"shells are not extended into void if fill density is 0";
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user