Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Issue] Sketcher angle constraint is not displayed where expected in 0.20.2 Windows LibPack build #8220

Closed
2 tasks done
Almisuifre opened this issue Jan 21, 2023 · 56 comments
Closed
2 tasks done
Assignees
Labels
Help wanted OS: Windows WB Sketcher Related to the Sketcher Workbench

Comments

@Almisuifre
Copy link

Almisuifre commented Jan 21, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Version

0.20 (Release)

Full version info

OS: Windows 10 Version 2009
Word size of FreeCAD: 64-bit
Version: 0.20.2.29177 +426 (Git)
Build type: Release
Branch: (HEAD detached from 0.20.2)
Hash: 930dd9a76203a3260b1e6256c70c1c3cad8c5cb8
Python 3.8.10, Qt 5.15.2, Coin 4.0.1, Vtk 8.2.0, OCC 7.6.3
Locale: French/France (fr_FR)

Subproject(s) affected?

Sketcher

Problem description

Hi,
I came to see you because I loaded the version 0.20.2.
I notice a nice bug positioning on the constraints of angles. See for yourself!
They seem unarmed attached to the central axis instead of the usual position. This is rather embarrassing because we no longer know what the constraint applies to.
image001
image
image002

The steps that lead to the bug:
Start FreeCAD 0.20.2
New tab
New body
Select a starting face (construction line)
Switch to sketcher mode
Draw a picture
Place the angle rib. It appears in the wrong place.

This produces a bad experience because we no longer know what the constraint is attached to. It seems to be frozen between the central axes and the angle we want to set. It seems to stand out from the selection.

Anything else?

https://discord.com/channels/870877411049357352/873944627806339082/1065907714242387978

Code of Conduct

  • I agree to follow this project's Code of Conduct
@luzpaz
Copy link
Contributor

luzpaz commented Jan 21, 2023

Can you please share an example file? (compress the .FCStd in to zip to be able to attach it to this ticket)

@Almisuifre
Copy link
Author

Okay, so here’s a video that shows the problem, as well as a file generated during the video.
Youtube Bug contraint
Drive>FreeCAD0.20.2

@adrianinsaval
Copy link
Member

I reattach here the file so we don't depend on external hosting (zip extension is a dummy, just rename)
TestContraintAngle-001.FCStd.zip

@adrianinsaval
Copy link
Member

I can reproduce this on 0.20.2 but not on 0.21

OS: Windows 10 Version 2009
Word size of FreeCAD: 64-bit
Version: 0.21.0.31641 (Git)
Build type: Release
Branch: master
Hash: d28d63b87b60161419c6c0c532fbbfaed96926b8
Python 3.10.8, Qt 5.15.6, Coin 4.0.0, Vtk 9.1.0, OCC 7.6.3
Locale: Spanish/Paraguay (es_PY)
Installed mods:

@Almisuifre
Copy link
Author

I reattach here the file so we don't depend on external hosting (zip extension is a dummy, just rename) TestContraintAngle-001.FCStd.zip
Okay, is that better?
TestContraintAngle-001.zip

I can reproduce this on 0.20.2 but not on 0.21

OS: Windows 10 Version 2009
Word size of FreeCAD: 64-bit
Version: 0.21.0.31641 (Git)
Build type: Release
Branch: master
Hash: d28d63b87b60161419c6c0c532fbbfaed96926b8
Python 3.10.8, Qt 5.15.6, Coin 4.0.0, Vtk 9.1.0, OCC 7.6.3
Locale: Spanish/Paraguay (es_PY)
Installed mods:

I haven’t tried with the last build. I seem to have heard that it didn’t have an impact on Linux. But now I’m on Windows!
I will try anyway-we never know. I come back to see to tell you what the situation is.

@Almisuifre
Copy link
Author

Ok, I tested on the 0.21 successfully. So this is a problem with the 0.20.2 version!
Here’s the video evidence.
Youtube not bug contraint in 0.21
Tested with the same workstation. Namely:
[code]
OS: Windows 10 Version 2009
Word size of FreeCAD: 64-bit
Version: 0.21.0.31641 (Git)
Build type: Release
Branch: master
Hash: d28d63b
Python 3.10.8, Qt 5.15.6, Coin 4.0.0, Vtk 9.1.0, OCC 7.6.3
Locale: French/France (fr_FR)
[/code]
Here is the file generated in a zip.
TestContraintAngle-002-FC021.zip
What more can I bring?

@Almisuifre
Copy link
Author

Hi there,
I don’t see a comeback. Are the investigations still ongoing? I haven’t seen a new stable version either. What about?

@adrianinsaval
Copy link
Member

Did this work correctly on 0.20.1? Someone would have to do a bisect to find out where the problem originated

@Almisuifre
Copy link
Author

Yes, it worked on version 0.20.1

@Almisuifre
Copy link
Author

Good morning,
For your information, yesterday I helped a beginner in FreeCAD 0.20.2. The behavior is strictly the same as at home. The angle constraint works but its position is not correct. It is also under Windows 10. We can therefore exclude the case of a single installation on a single station.
Thank you so much for your help.

@Syres916
Copy link
Contributor

Syres916 commented Feb 6, 2023

@adrianinsaval I really don't think that a Git Bisect is required in this instance, I've proved that this is a Windows LibPack specific issue unless you believe there's some code that's changed and only affected the LibPack build. The following is what I wrote in another report on the forum and I've advised Uwe to please check it on the next LibPack build which version that may be:

this regression is not a general 0.20.2 regression as possibly first thought, it's specific to the Windows Libpack build. I'd thought to myself "Why have I not seen this on my Conda Linux build?" (I use 0.20.2 on a daily basis) because it doesn't occur. It also doesn't occur on the same system with the same config files using the Conda Windows build, it only occurs in the Libpack build

@Syres916
Copy link
Contributor

Syres916 commented Feb 6, 2023

Could the title if this issue please be changed to something like [Issue] Sketcher angle constraint is not displayed where expected in 0.20.2 Windows LibPack build

@Almisuifre Almisuifre changed the title [Problem] Constraints of angles [Problème] La contrainte d’angle Sketcher n’est pas affichée comme prévu dans la version 0.20.2 de Windows LibPack Feb 6, 2023
@Almisuifre
Copy link
Author

Hello @Syres916, thank you for returning.

I changed the title, as you suggested.

I found the bug in other windows. Probably we have the same LibPack. In this case, would it not be possible to test on our machine, then switch to the correct adhequat version to regain optimal operation.
I’m talking about a minor change in the code (if LbPack W10, then old version/ Otherwise, take the new version). That would avoid the problem.

Another solution, providing an equivalent to LibPack can be by switching to Conda including for windows.

Finally, well, first and foremost is to have seen what happens ;)

@Almisuifre Almisuifre changed the title [Problème] La contrainte d’angle Sketcher n’est pas affichée comme prévu dans la version 0.20.2 de Windows LibPack [Issue] Sketcher angle constraint is not displayed where expected in 0.20.2 Windows LibPack build Feb 6, 2023
@ghost
Copy link

ghost commented Feb 17, 2023

I'm using Windows 10 and have had the same experience. I uninstalled, went to version 20.1, NO problems there. Re-downloaded 20.2, unlucky, problem is permanent. Verified on 2 different machines running different versions of windows 10. One is a laptop (Asus), the other is a generic tower with a Nvidia RTX20.. video board.
Cheers, p.

@Almisuifre
Copy link
Author

Thank you very much for your repport. It is a contribution. We must now count on the developers to find a way out.

@luzpaz
Copy link
Contributor

luzpaz commented Feb 17, 2023

@donovaly tagging you temporarily on this if that's OK, since it's libpack related

@ghost
Copy link

ghost commented Feb 17, 2023 via email

@FreeCAD FreeCAD deleted a comment Feb 18, 2023
@donovaly
Copy link
Member

@donovaly tagging you temporarily on this if that's OK, since it's libpack related

it seems it is not Libpack-related: https://forum.freecad.org/viewtopic.php?p=658814#p658814

And if it would be, what should I do? What library is causing this?

@donovaly
Copy link
Member

@abdullahtahiriyo , do you have an idea what could cause this regression bug.
As it seems not to be a problem with the libraries in the Libpack, it might be something else that was changed between FC 0.20.1 and 0.20.2.

@ghost
Copy link

ghost commented Feb 18, 2023 via email

@donovaly
Copy link
Member

donovaly commented Feb 22, 2023

I'll send you what I just said to Abdullah, and if you permit, I'll send Abdullah your comment. Cheers, p.

Cheers as well but please don't pollute the GitHub comments with useless statements. Please also don't make full cites, only the parts you refer to.
I also don't understand why you advertise BlueMail for Android. This might be a nice app but has nothing to do with the issue. Maybe it is better if you login to GitHub to join the conversation more properly.

@abdullahtahiriyo
Copy link
Contributor

@donovaly

We must be doing something different.

The easiest, if you download the file and open it. Do you get the issue in the video?

I get this:
imagen

@donovaly
Copy link
Member

I get this:

I get this with today's master and also 0.20.2:

FreeCAD_KnJfixb6Z4

So I can confirm the bug and also that with 0.20.1 everything is fine.
Give me a day to check to compile with the LibPack we used for 0.20.1 onto master. Then i can say for sure if the LibPack is to blame or not.

@abdullahtahiriyo
Copy link
Contributor

@adrianinsaval

Adrian, I wonder if you could help me out here. Could you please confirm that in Linux master we do not get this issue?

If we got it, then it would save Uwe some quite compiling time (which appears to be pointless if the difference is really just a SSL fix) and we could do a git bisect to track the PR.

I mean if you have time, of couse.

@abdullahtahiriyo
Copy link
Contributor

@adrianinsaval

Sorry. Do not bother. I have finished the PR I was handling and I am taking care now.

@abdullahtahiriyo
Copy link
Contributor

@donovaly

I have just built today's master and I do not get the artifact.

OS: Ubuntu 20.04.5 LTS (MATE/mate)
Word size of FreeCAD: 64-bit
Version: 0.21.0.31995 (Git) AppImage
Build type: Debug
Branch: master
Hash: b2ee277
Python 3.8.10, Qt 5.12.8, Coin 4.0.0, Vtk 7.1.1, OCC 7.5.2

@FreeCAD FreeCAD deleted a comment Feb 25, 2023
@GrzegorzKrug
Copy link

I got same issue. Maybe this could help you solve it.
obraz

[code]
OS: Windows 10 Version 2009
Word size of FreeCAD: 64-bit
Version: 0.20.2.29177 +426 (Git)
Build type: Release
Branch: (HEAD detached from 0.20.2)
Hash: 930dd9a
Python 3.8.10, Qt 5.15.2, Coin 4.0.1, Vtk 8.2.0, OCC 7.6.3
Locale: English/United Kingdom (en_GB)
[/code]

attachment sketch:
angle.zip

@Bebop182
Copy link

Bebop182 commented Mar 3, 2023

@GrzegorzKrug it is already solved un the latest 0.21 dev build

@ghost
Copy link

ghost commented Mar 3, 2023 via email

@ghost
Copy link

ghost commented Mar 4, 2023

Oops, sorry for premature praise, I've downloaded and tested, bug still stuck in relay contacts!
Cheers, p.

@Bebop182
Copy link

Bebop182 commented Mar 4, 2023

I just tested the latest dev installer commit 32110 and the bug is there. But I'm running the previous non installer variant commit 31917 and the bug is not present, So it was either reintroduced or built differently.
Here's last week build if you want to try it out:
https://drive.google.com/file/d/1miMmSk-1zLebMTR10Fnukd44Nky-rtvv/view?usp=share_link

@Almisuifre
Copy link
Author

Hi,
I just downloaded the weekly version. I installed it and tested it.

Here is my report. Everything is going well. I no longer encounter the bug I had before. So I propose to switch fairly quickly to the branch 0.21.0.

no-bug-weekly

https://drive.google.com/file/d/1JkqnShrq6Cbw-1MRSQziMloj5yo8r4TJ/view?usp=share_link

Anyway, great work.

@adrianinsaval
Copy link
Member

So I can confirm the bug and also that with 0.20.1 everything is fine. Give me a day to check to compile with the LibPack we used for 0.20.1 onto master. Then i can say for sure if the LibPack is to blame or not.

Where you able to do this @donovaly?

@FreeCAD FreeCAD deleted a comment Mar 9, 2023
@FreeCAD FreeCAD deleted a comment Mar 9, 2023
@FreeCAD FreeCAD deleted a comment Mar 9, 2023
@donovaly
Copy link
Member

donovaly commented Mar 9, 2023

I don't had time yet. Testing if and how the Libpack is affected take at least a whole work day.

@donovaly
Copy link
Member

Wow, after just 4 hours of testing I found the reason and a fix:

  • the LibPack is not to blame
  • the bug is in Visual Studio since version 17.4 (FC 0.20.1 was compiled using version 17.3.x)
  • the "fix" is to disable the compiler optimization for the build target "SketcherGui" (default is /O2 and also setting /O1 fails)

I am preparing a new build for FC 0.21.dev right now and will inform you when it is online.
Then I will also make a new binary for FC 0.20.2, but this needs some hours, this the master build comes first.

@donovaly
Copy link
Member

Important info: Don't update to version 17.5 of Visual studio because it fails completely due to a bug with boost. The Microsoft guys wrote 2 days ago that they found a fix and will release version 17.5.2 in a few days. Then will also check if the compiler optimization issue we have here is fixed too because in the forum people wrote that the boost issue also only appears with compiler optimization turned on.

@abdullahtahiriyo
Copy link
Contributor

@donovaly
That is indeed a good catch. We tend to think compilers are never the problem. Most often they are not. Yet you got your black swan.

@donovaly
Copy link
Member

For those who are interested:
Here is a new Windows build of latest master: https://github.com/FreeCAD/FreeCAD-Bundle/releases/download/weekly-builds/FreeCAD-0.21.0dev-32258-Installer.exe

@donovaly
Copy link
Member

@donovaly
Copy link
Member

The new installer is online since some days now -> closing as fixed.

@ghost
Copy link

ghost commented Mar 21, 2023 via email

@FreeCAD FreeCAD deleted a comment Mar 22, 2023
@Almisuifre
Copy link
Author

Hi,

I just saw your messages. Sorry for late mojn. Life is so full of challenges, I can’t be everywhere.

I tested the dev version, it works perfectly. Bug fixed.

I am now taking the official version 0.20.3 to continue my work.

Thank you very much for your patience and also for finding a way out. Thanks to the whole team, it’s great to be able to help and fix what needs to be done.

@FreeCAD FreeCAD deleted a comment Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help wanted OS: Windows WB Sketcher Related to the Sketcher Workbench
Projects
None yet
Development

No branches or pull requests

8 participants