-
Notifications
You must be signed in to change notification settings - Fork 511
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
coexistence of "In Workplane" and "Anywhere in 3d" #197
Comments
Here is a shorter sequence:
Note that, if we leave out the assembly step, we just get "In Workplane", as expected. |
@wpwrak, It seems I can't reproduce it on Windows. And it's strange how the following code can cause this:
|
@Schroedinger, can you explain it? |
Seems like UB... |
UB = Ubuntu ? Yes, I'm on Ubuntu 16.10. I can collapse the quantum state (i.e., make it check only "Anywhere in 3d") by |
@wpwrak, What build of SolveSpace you use on Yakety now? Is it possible run it on Xenial? |
@Symbian9: I just build from sources, with a number of fixes and improvement, either from @Evil-Spirit or self-made, that are not in mainline. Not sure about Xenial. |
@wpwrak Are you still able to reproduce this? I've refactored a lot of the UI code in the meantime. |
Have not been able to reproduce for two years, and no response in two months, closing. |
I was able to reproduce it.
|
OK, I figured it out. This is caused by rather obnoxious GTK behavior where set_active is not independent from causing a trigger event. |
@phkahler I'm pretty sure this is not fixed--the underlying issue is the behavior of |
What do you mean? On the surface this seems to fix it. Are you saying the condition may reappear? |
I don't know how to trigger it, but there is still a latent bug left in the GTK UI code, yes. |
It it possible to make the "Sketch" menu show that both "In Workplane" and "Anywhere in 3d" are selected. Schroedinger would approve :-)
I've seen this on several occasions. Here is a somewhat circuitous way to reproduce it. I'm still looking for a simpler one.
solvespace master (f29d9d7) plus small local changes
Let's use this example from https://gitlab.com/anelok/mexp.git master (3acc289)
The text was updated successfully, but these errors were encountered: