-
Notifications
You must be signed in to change notification settings - Fork 65
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
Release v15.3 #611
Release v15.3 #611
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@a-corni you should likely use MyPy v1.6.1 in order to make the tests pass. |
MyPy version is kept at 1.2... I will keep investigating 🔍 |
The error seems to be coming from the module referencing. They made a new release on Nov. 14th and the error shown (see below) points directly to it according to this issue on mypy
I have followed what the mypy developpers suggested there: Add to mypy.ini:
But this does not seem to have changed anything - which is weird, it should have deleted the error messages coming from referencing... |
Solved by using |
Making a hot fix to enable the use of parametrized phase in eom mode.