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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue with UMAP import #153

Closed
wants to merge 1 commit into from
Closed

Fix issue with UMAP import #153

wants to merge 1 commit into from

Conversation

xoubinha
Copy link

  • Modify UMAP import.

Explanation:
When trying to import bertopic I was getting an AttributeError related to UMAP import, saying:
module umap has no attribute UMAP
I checked that I had the correct version of umap-learn but still it wasn't working. Changing the imports as in the PR solved the issue. 馃槉

@MaartenGr
Copy link
Owner

The changes that you propose are workarounds to the issues mentioned here and here. To understand whether the change is truly necessary, it is important that we understand why you are facing this issue. Did you have a previous version of umap installed together with umap-learn? That typically seems to be the issue as many install UMAP with pip install umap instead of pip install umap-learn. Also, did you try it out in a fresh environment?

@xoubinha
Copy link
Author

I did it in a fresh environment, but it must have used already installed packages. I tried it again uninstalling everything and it did work.

@xoubinha xoubinha closed this Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants