[PD] two bugs when creating a multi-transform feature

Post here for help on using FreeCAD's graphical user interface (GUI).
Forum rules
and Helpful information
IMPORTANT: Please click here and read this first, before asking for help

Also, be nice to others! Read the FreeCAD code of conduct!
User avatar
uwestoehr
Veteran
Posts: 4961
Joined: Sun Jan 27, 2019 3:21 am
Location: Germany
Contact:

[PD] two bugs when creating a multi-transform feature

Post by uwestoehr »

In the fresh FC 0.19 release I encounter two bugs with multi-transform features:

Bug A
1. select e.g. a Pad
2. add a multi-transform feature to it
3. add in the transform dialog a transformation of your choice, e.g. a scaled transformation

result: the transformation succeeded but you suddenly see nothing anymore in the model view

Bug B:
4. Cancel the multi-transform dialog

result: the operation is not canceled but you end up with 2 orphaned new features in the model tree

Here is a screencast showing both bugs:
PLJtfMIooA.gif
PLJtfMIooA.gif (386.57 KiB) Viewed 1251 times

OK to file bug reports?


OS: Windows 10 Version 2009
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.24276 (Git)
Build type: Release
Branch: master
Hash: 2599d67c2bd41a5f051b994b6da7890a2b4ecc4c
Python version: 3.8.6+
Qt version: 5.15.1
Coin version: 4.0.1
OCC version: 7.5.0
Locale: German/Germany (de_DE)
Bance
Veteran
Posts: 4038
Joined: Wed Feb 11, 2015 3:00 pm
Location: London

Re: [PD] two bugs when creating a multi-transform feature

Post by Bance »

A scaled may not be the first transform.

PartDesign_Scaled
drmacro
Veteran
Posts: 8637
Joined: Sun Mar 02, 2014 4:35 pm

Re: [PD] two bugs when creating a multi-transform feature

Post by drmacro »

I was just attempting mirror in PD in today's PPA.

Tried a face, a datum line, and a datum plane. No errors, just no mirror.

Works fine if the object is the result of a single operation. But, add a pocket. And no mirror, no error.

Maybe it's me. :oops: :mrgreen:

Body and Body001 should have similar mirrored look.
Snip macro screenshot-a7acb1.png
Snip macro screenshot-a7acb1.png (246.82 KiB) Viewed 1214 times
Attachments
MirrorNot.FCStd
(43.88 KiB) Downloaded 30 times
Star Trek II: The Wrath of Khan: Spock: "...His pattern indicates two-dimensional thinking."
Bance
Veteran
Posts: 4038
Joined: Wed Feb 11, 2015 3:00 pm
Location: London

Re: [PD] two bugs when creating a multi-transform feature

Post by Bance »

I don't have latest, but if you mirror a pocket into naught you'll get naught, select pad and pocket and the op succeeds.

OS: Linux Mint 20.1 (XFCE/xfce)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.24212 (Git) AppImage
Build type: Release
Branch: master
Hash: 3d227cf7179c00e57f2733f947b4b1b0368d219a
Python version: 3.8.6
Qt version: 5.12.5
Coin version: 4.0.0
OCC version: 7.4.0
Locale: English/United Kingdom (en_GB)
drmacro
Veteran
Posts: 8637
Joined: Sun Mar 02, 2014 4:35 pm

Re: [PD] two bugs when creating a multi-transform feature

Post by drmacro »

I don't get that?

Isn't the tip the current state of the solid, why do i need to select previous parts of the tree?

that means if I have a solid that is the cumulative result of, intermixed, 3 pads and 3 pockets I have to select 6 things to mirror the cumulative soild?

:?:
Star Trek II: The Wrath of Khan: Spock: "...His pattern indicates two-dimensional thinking."
Bance
Veteran
Posts: 4038
Joined: Wed Feb 11, 2015 3:00 pm
Location: London

Re: [PD] two bugs when creating a multi-transform feature

Post by Bance »

Because it's a feature transform not a body transform.
drmacro
Veteran
Posts: 8637
Joined: Sun Mar 02, 2014 4:35 pm

Re: [PD] two bugs when creating a multi-transform feature

Post by drmacro »

Ok, I guess I can grok that.

But, in the case I posted, there is no indication that the mirror failed on Pocket. It just proceeds to add a Mirrored feature to the tree.

That is really my point. No error or warning. And the Mirrored feature passes geometry check.

So, what is it? Is there now two Pocket features, one just happens to be labeled Mirrored...but isn't mirrored.

It's ghost geometry, anti-matter. :mrgreen:
Star Trek II: The Wrath of Khan: Spock: "...His pattern indicates two-dimensional thinking."
Bance
Veteran
Posts: 4038
Joined: Wed Feb 11, 2015 3:00 pm
Location: London

Re: [PD] two bugs when creating a multi-transform feature

Post by Bance »

I remember there was some discussion about this, It was pointed out that it was unintuitive for an over extended transform to simply fail,

and that the failed transform feature did no harm, so it was given that the transform would succeed but the extra features ignored.

It is actually pointed out in the wiki, although I think the entry needs a little love.
drmacro
Veteran
Posts: 8637
Joined: Sun Mar 02, 2014 4:35 pm

Re: [PD] two bugs when creating a multi-transform feature

Post by drmacro »

By over extended do mean it would create non-contigous solid?

In the context of mirroring a feature that creates two non-contigous solids, not failing and created a phantom feature seems like a kludge. ...to me. especially when the chosen feature happens to be a negative.

The other question is, as was the case I wanted here, how to mirror the tip (i.e. the current soild), not the feature.
I guess selecting all the features that constitute the tip...???🤔 That seems it could get dicey if there are a lot of features.

And, of course, mirroring tip around a face on the tip is just going invite the topo monster. :roll:
Star Trek II: The Wrath of Khan: Spock: "...His pattern indicates two-dimensional thinking."
Bance
Veteran
Posts: 4038
Joined: Wed Feb 11, 2015 3:00 pm
Location: London

Re: [PD] two bugs when creating a multi-transform feature

Post by Bance »

drmacro wrote: Sun Feb 28, 2021 12:02 am By over extended do mean it would create non-contigous solid?
Yes, or a hole in the air (like yours :lol: )

So, iirc it was failing without error before the distance was entered (corrected) Etc. I agree, it is not an ideal solution.

If you want to mirror the entire body you must use Part mirror (with the usual caveats.)
Post Reply