Page 1 of 1

PR #3788 bind expression to pad offset

Posted: Sat Aug 08, 2020 2:09 am
by uwestoehr
Having an expression also for the pad offset, but this works fine (with the PR applied), see this example:
OffsetTest.FCStd
(27.08 KiB) Downloaded 165 times

Re: PR #3788

Posted: Sat Aug 08, 2020 6:02 am
by openBrain
As per the guidelines, this isn't needed to post here for any PR you submit, especially these like this one that don't need any discussion.
In any case, please at least add something descriptive in the title and provide a link to the PR in the OP.

Re: PR #3788

Posted: Sat Aug 08, 2020 7:47 am
by chrisb
Can you explain this further? The pad offset was here expression aware before.

OS: macOS High Sierra (10.13)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.22164 (Git)
Build type: Release
Branch: master
Hash: f2a9f712e8d68ee2393d0fe748c02096c4148a32
Python version: 3.8.5
Qt version: 5.12.5
Coin version: 4.0.0
OCC version: 7.4.0
Locale: C/Default (C)

Re: PR #3788

Posted: Sat Aug 08, 2020 7:54 am
by chrisb
I saw that openBrain responded already...
I just checked: the pad offset was already expression aware in 0.18.
openBrain wrote: Sat Aug 08, 2020 6:02 am In any case, please at least add something descriptive in the title and provide a link to the PR in the OP.
That's better than nothing, but not helpful for average users.

Re: PR #3788

Posted: Sat Aug 08, 2020 8:30 am
by openBrain
chrisb wrote: Sat Aug 08, 2020 7:47 am Can you explain this further? The pad offset was here expression aware before.
This is in the property editor, but not in the task dialog at creation time. ;)
chrisb wrote: Sat Aug 08, 2020 7:54 am That's better than nothing, but not helpful for average users.
Sure. I think people interested in can browse the Github PRs directly.

Re: PR #3788

Posted: Sat Aug 08, 2020 11:49 am
by uwestoehr
openBrain wrote: Sat Aug 08, 2020 6:02 am this isn't needed to post here for any PR you submit, especially these like this one that don't need any discussion.
In any case, please at least add something descriptive in the title and provide a link to the PR in the OP.
As you see I only do report here, if I think it is necessary. In this case I wanted to give an example file, so I needed to upload it and Github did not allow me this (only as ZIP).
Sorry that I this time forgot to give the thread a clear title.