PR #3748 allow multiselection on creation of pattern
PR #3748 allow multiselection on creation of pattern
While designing with FC last week it annoyed me that to get patterns of more than one feature, I am forced to create a pattern using only one feature. Then in a second step I had to add the other features to the pattern.
Today I had a look and enabled that one can select several features on creation of a pattern. So with this PR this is now possible:
I tested the feature thoroughly (using several bodies, combination of features etc.) and I think it is stable.
Today I had a look and enabled that one can select several features on creation of a pattern. So with this PR this is now possible:
I tested the feature thoroughly (using several bodies, combination of features etc.) and I think it is stable.
Last edited by uwestoehr on Mon Jul 27, 2020 9:45 am, edited 1 time in total.
Re: PR #3748 allow multiselection on creation of pattern
Very helpful! Thanks.
In rare cases a reordering is necessary. Would it be possible to include? If not as workaround the old mechanism can be used, of course.
In rare cases a reordering is necessary. Would it be possible to include? If not as workaround the old mechanism can be used, of course.
A Sketcher Lecture with in-depth information is available in English, auf Deutsch, en français, en español.
Re: PR #3748 allow multiselection on creation of pattern
The stability of the pattern is completely out of the range of my thought. I was rather thinking about buttons "Move up", "Move down" which would move a selected feature up or down the list.
The order of the features can be significant.
A Sketcher Lecture with in-depth information is available in English, auf Deutsch, en français, en español.
Re: PR #3748 allow multiselection on creation of pattern
Do you think so or you have tested this? A testcase would be nice to have. You can create one since my patch does not introduce the multifeature patterning, it just updates the UI that you can do it directly without the need of creating the pattern first then adding more features to it.
I tested the feature order and cannot see any difference on how the features in the pattern are ordered. To break the pattern, you could at first create a pattern and then move one if its features behind the pattern. Then you will get of course an error.
Re: PR #3748 allow multiselection on creation of pattern
I'm afraid I haven't made myself clear. A testcase should help, but it may have to wait until tonight.
A Sketcher Lecture with in-depth information is available in English, auf Deutsch, en français, en español.
Re: PR #3748 allow multiselection on creation of pattern
Here is the example (didn't have access to FreeCAD sooner). I have a Cube, a padded circle and a rectangular pocket:
Making a polar pattern with Pad before Pocket:
And here with Pocket before Pad:
The version with Pad before Pocket is attached.- Attachments
-
- pad_before_pocket.FCStd
- (22.08 KiB) Downloaded 182 times
A Sketcher Lecture with in-depth information is available in English, auf Deutsch, en français, en español.
Re: PR #3748 allow multiselection on creation of pattern
Part of your PR is this commit: https://github.com/FreeCAD/FreeCAD/pull ... 68f133d0ea
What's the point about it? The commit message is "Merge branch 'master' of github.com:FreeCAD/FreeCAD into PD-patterning". Apparently it's not something you developed but the code is not part of master either. So, where does it come from?
And according to your description the problem is/was a pure GUI thing so I wouldn't expect any changes on feature level.
Re: PR #3748 allow multiselection on creation of pattern
I think such messages appear when the developer has internal branches that he merges before opening the pull request. Git merge always seems to create a commit like this, even if it just says "merging this branch to another".
Always add the important information to your posts if you need help. Also see Tutorials and Video tutorials.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
To support the documentation effort, and code development, your donation is appreciated: liberapay.com/FreeCAD.
Re: PR #3748 allow multiselection on creation of pattern
With git commit 94ca4388e you can move the items directly with the mouse.