[solved] Value increase error: editing 3D Surface changes values all by itself
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Be nice to others! Respect the FreeCAD code of conduct!
[solved] Value increase error: editing 3D Surface changes values all by itself
Hi Russel,
a old Problem comes back to 3D Surface OP:
If i reopen the UI for the OP, some values gets increased, here the value for the Sample Intervall und BounBox extra offset.
Atm i'm on
OS: openSUSE Leap 15.1 (KDE//usr/share/xsessions/plasma5)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20052 (Git) AppImage
Build type: Release
Branch: master
Hash: 6291921e9abe737e4440ff5814ca58547defadef
Python version: 3.8.2
Qt version: 5.12.5
Coin version: 4.0.0
OCC version: 7.4.0
Locale: German/Germany (de_DE)
a old Problem comes back to 3D Surface OP:
If i reopen the UI for the OP, some values gets increased, here the value for the Sample Intervall und BounBox extra offset.
Atm i'm on
OS: openSUSE Leap 15.1 (KDE//usr/share/xsessions/plasma5)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20052 (Git) AppImage
Build type: Release
Branch: master
Hash: 6291921e9abe737e4440ff5814ca58547defadef
Python version: 3.8.2
Qt version: 5.12.5
Coin version: 4.0.0
OCC version: 7.4.0
Locale: German/Germany (de_DE)
Last edited by herbk on Tue Apr 07, 2020 11:20 am, edited 1 time in total.
Gruß Herbert
Re: 3D Surface - Face Selection (New feature) [Merged]
Morning Sir,
I could use some more information about the changes to better troubleshoot the issue. What was the initial value? What is the new value that FreeCAD used in the UI? Is this within an initial operation, or editing of an existing operation? I do know we have some GUI value issues reported elsewhere here in the Path/CAM forum.
And, it is possible we have a bug somewhere in the GUI side of the PathWB.
BR
Russ
I could use some more information about the changes to better troubleshoot the issue. What was the initial value? What is the new value that FreeCAD used in the UI? Is this within an initial operation, or editing of an existing operation? I do know we have some GUI value issues reported elsewhere here in the Path/CAM forum.
Looking through the entirety of the updated PathSurface module code, here is what I see:
- The SampleInterval value is only re-assigned if it is below 0.001 millimeters or above 25.4 millimeters. I will change the lower limit to 0.0001 millimeters in the next PR. So, if the user has entered values outside of these limits, the user's value is changed to the nearest limit value, minimum or maximum depending on which limit was violated.
- The DropCutterExtraOffset value is never re-assigned within the code. So, if this value is being changed, it is not explicitly done so in the PathSurface module.
And, it is possible we have a bug somewhere in the GUI side of the PathWB.
BR
Russ
Re: 3D Surface - Face Selection (New feature) [Merged]
Hi Russ,
OK...here with some pics:
i set the values for BoundBox extra offset and Sample Interval to 1mm and close the UI by clicking OK
I reopen the UI by doubteclick the Operation and get this values shown. Closing the UI by OK shows the path calculetet by this values.
reopen the UI shows again increased values...
My number of decimals is only at 2 and also the precision of path calculations is only at 0.01mm
OK...here with some pics:
i set the values for BoundBox extra offset and Sample Interval to 1mm and close the UI by clicking OK
I reopen the UI by doubteclick the Operation and get this values shown. Closing the UI by OK shows the path calculetet by this values.
reopen the UI shows again increased values...
My number of decimals is only at 2 and also the precision of path calculations is only at 0.01mm
Gruß Herbert
Re: 3D Surface - Face Selection (New feature) [Merged]
Morning Sir,
Wowser! That is quite the error. That looks like someone messed with the units conversion code or something deeper within FreeCAD. The error is clearly increasing the value by a factor of 10 each time. Are you seeing the same error in other operations?
Russ
Wowser! That is quite the error. That looks like someone messed with the units conversion code or something deeper within FreeCAD. The error is clearly increasing the value by a factor of 10 each time. Are you seeing the same error in other operations?
Paging
,sliptonic wrote:@Sliptonic
, andmlampert wrote:@MLampert
. Any ideas, Gentlemen?vocx wrote:@Vocx
Russ
Re: 3D Surface - Face Selection (New feature) [Merged]
Hi Russ,
No, i don't, - but i know BoundBox extra offset and Sample Interval only from 3D Surfacing OP...
Gruß Herbert
Re: 3D Surface - Face Selection (New feature) [Merged]
Strange. I am not getting this behavior in 3D Surface on my Windows 10 machine. I used the FC version below, and checked on Version: 0.19.20167 also. No errors as you have posted. Perhaps
can check his behavior. He uses MacOS.chrisb wrote:@ChrisB
Russ
OS: Windows 10 (10.0)
Word size of OS: 64-bit
Word size of FreeCAD: 64-bit
Version: 0.19.20036 (Git)
Build type: Release
Branch: master
Hash: 953ae1e6e917fa6860564c80fdc1f20950a5c0ac
Python version: 3.6.8
Qt version: 5.12.1
Coin version: 4.0.0a
OCC version: 7.3.0
Locale: English/United States (en_US)
Re: 3D Surface - Face Selection (New feature) [Merged]
I don't remember a change to the core code in recent days that could affect this. I mean, there were some changes to Base::Quantity, but these were done a few weeks ago. Did you not have problems until very recently?
I'm not familiar enough with Path's code, but it seems the "boundBoxExtraOffsetX" item in the GUI is responsible.
Inside the PathSurfaceGui.TaskPanelOpPage, it is assigned to a new object.
Code: Select all
obj.DropCutterExtraOffset.x = FreeCAD.Units.Quantity(self.form.boundBoxExtraOffsetX.text()).Value
Code: Select all
Command = PathOpGui.SetupOperation(...)
Maybe check manually the value of self.form.boundBoxExtraOffsetX with print()s.
You also have this
Code: Select all
self.form.boundBoxExtraOffsetX.setText(str(obj.DropCutterExtraOffset.x))
Code: Select all
print(self.form.boundBoxExtraOffsetX.text())
print(obj.DropCutterExtraOffset.x)
...
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: 3D Surface - Face Selection (New feature) [Merged]
Herbert, can you please upload the model where you are facing these issues? I created a cone and used your parameters - now FreeCAD is busy since 10 minutes now and I would not like to create another and face the same delay ...
A Sketcher Lecture with in-depth information is available in English, auf Deutsch, en français, en español.
Re: 3D Surface - Face Selection (New feature) [Merged]
Hi Chris,
no prob, is attached.
It's not a new problem, a made checks down to Appimage release .19518, the oldest i have.
I think to remember we have had this prob at other OPs/parameters to, but that was several month in the past...
no prob, is attached.
It's not a new problem, a made checks down to Appimage release .19518, the oldest i have.
I think to remember we have had this prob at other OPs/parameters to, but that was several month in the past...
- Attachments
-
- Spulen_Pressform.FCStd
- (223.56 KiB) Downloaded 104 times
Last edited by herbk on Sun Mar 22, 2020 7:37 pm, edited 1 time in total.
Gruß Herbert