Add ScopeReducerExplorer #35
Conversation
roquelopez
force-pushed
the
add_scope_explorer
branch
from
63ff1ed to
e53df3a
Compare
|
@viguardieiro I made some changes to include the explorer as part of the API. Feel free to merge it, thanks! |
requirements.txt
Outdated
| @@ -8,3 +8,9 @@ scikit-learn | |||
| tabulate | |||
| flair | |||
| requests | |||
| scipy==1.10.1 | |||
There was a problem hiding this comment.
Do we need to pin the exact version of all of these libraries? Couldn't we just specify the minimum and/or maximum versions that work with our code? I'm afraid that pinning very common libraries like scipy and matplotlib will cause conflicts with other libraries.
There was a problem hiding this comment.
In PR #37, I was able to fix the errors by using just:
scipy<1.13
matplotlib<3.9
The root cause of the errors is in the packages PolyFuzz and gensim, which are transitive dependencies.
There was a problem hiding this comment.
Yes, I agree, it's better to have flexible versions. I relaxed the versions of the other dependencies.
The API import in the example was still referring to the old module, I added a small fix for that. |
roquelopez
force-pushed
the
add_scope_explorer
branch
from
55cb90d to
c2e898a
Compare
No description provided.