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

Drag-from-widget to add next widget constantly produces error messages #6410

Closed
wvdvegte opened this issue Apr 13, 2023 · 6 comments
Closed
Labels
bug A bug confirmed by the core team

Comments

@wvdvegte
Copy link

wvdvegte commented Apr 13, 2023

What's wrong?
Since recently, I can no longer easily drag a connector from a widget and select a next widget in the contextual menu. The first time directly after dragging, an Unexpected Error window comes up, with a report that can be submitted. Next, a larger error message window pops up. If I click Ignore, I can finally select the next widget. Every next time I want to add new widgets this way, two of these large-size error message windows pop up consecutively. After clicking Ignore on each, again I can proceed.

It appears this started happening since (1) I updated Mac OS to Ventura 13.3.1 and (2) I updated the Text mining add-on to v. 1.12.2

How can we reproduce the problem?
See attached movie:
Screen Recording 2023-04-13 at 15.42.50.mov.zip

What's your environment?

  • Operating system: Mac OS to Ventura 13.3.1 (Silicon)
  • Orange version: 3.34.1
  • How you installed Orange: from DMG

edit: updated add-on was text mining, not timeseries.

@wvdvegte wvdvegte added the bug report Bug is reported by user, not yet confirmed by the core team label Apr 13, 2023
@janezd janezd self-assigned this Apr 14, 2023
@janezd
Copy link
Contributor

janezd commented Apr 14, 2023

Screenshot 2023-04-14 at 15 19 43

Does updating the text mining add-on resolve the problem?

It seems that a part of text mining add-on expects a different version of numpy - which came from whoever knows - and fails at import. Does anyone know what's going on? @markotoplak, @PrimozGodec?

This resurrects the topic of pinning: Orange gives no warning if an add-on does not support a recent version of some library. One add-on (or Orange itself) may update pandas and an add-on may crash until the user updates it.

@ales-erjavec, beyond this, a crashing add-on can cause general problems in canvas. Could it be made more resilient?

@wvdvegte, thanks for this report. The concrete problem itself may be trivial to solve, but you warned us about a nasty general problem.

@wvdvegte
Copy link
Author

Hm, I confused text mining and timeseries, indeed. I uninstalled text mining, then reinstalled it. That did not help, unfortunately.

@wvdvegte
Copy link
Author

Update: I didn't discover at first, but I completely lost the text add-on after trying to reinstall it after uninstalling it as reported above. So far, I haven't been successful in getting it back.
I will open a separate issue for it.

@janezd janezd removed their assignment Apr 19, 2023
@janezd janezd added bug A bug confirmed by the core team and removed bug report Bug is reported by user, not yet confirmed by the core team labels Apr 19, 2023
@PrimozGodec
Copy link
Contributor

PrimozGodec commented Apr 25, 2023

@wvdvegte, sorry that it took us long to review your error. I tried to reproduce the error but could not (I have the same version of MacOS, Orange, and Text addon). Can you report to us the versions of the Python packages that you have installed?

You get versions by opening the Terminal and typing /Applications/Orange3.app/Contents/MacOS/pip freeze if you have Orange installed in the Applications folder. Change the path if your Orange is installed somewhere else.

@wvdvegte
Copy link
Author

@PrimozGodec "unfortunately", the problem disappeared. After uninstalling the text add-on as suggested above, and not being able to get it back, I uninstalled Orange and all plugins, then reinstalled everything as reported in #6416. This also eliminated the drag-from-widget problem. My apologies for not reporting accordingly!

@PrimozGodec
Copy link
Contributor

@wvdvegte, thank you for reporting and helping us. We spotted the error. It turned out that Gensim (one of the Text's dependencies) made a mistake in building the package with too new Numpy in version 4.3.1. We temporarily fixed it by omitting the problematic version biolab/orange3-text#967. I also proposed the PR that will fix the problem in their repository's next version of Gensim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug confirmed by the core team
Projects
None yet
Development

No branches or pull requests

3 participants