-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
🎣🏆 Repo cleanup and fix RGCN's hpo_default #1370
Conversation
the error seems to be equivalent to #1363, and this is something to fix in upstream https://github.com/cthoyt/class-resolver EDIT: there is already an issue for this: cthoyt/class-resolver#58 |
New v0.4.3 release of |
@mberr can we do a separate PR where we bump the minimum python version? Should we say py310 for now and be relatively progressive? |
I would be fine with it (3.10 is also the default Python version in the current Ubuntu LTS). Is there any reason why we need to bump the version? |
(cherry picked from commit 0500ce6)
Fix #1367
Also does some repo cleanup due to new versions of black & mypy.
Also fix #1363 by increasing the minimum class-resolver version.