Adapted test cases to Sympy Version 1.6
Due to Sympy Version 1.6 the sp.Abs function in the test_interpolation test cases could not be identified as integer anymore. This is now fixed by not checking the type of sp.Abs anymore but by checking its argument.
Furthermore depreciation warning fixes of pandas and pycuda and pytest and adaption to new flake8 Version.
Merge request reports
Activity
assigned to @hoenig
113 113 return self.__repr__() 114 114 115 115 def __repr__(self): 116 116 repr = 'if:({!r}) '.format(self.condition_expr) changed this line in version 3 of the diff
Isn't it fairly common to implement the
__repr__
function for a class?https://stackoverflow.com/questions/1436703/difference-between-str-and-repr
https://www.journaldev.com/22460/python-str-repr-functions
Or do I misinterpret something?
I also didn't understand Jan when I first read this.
I can explain Jan for you:
repr
(line 116) is a builtin (he was not referring to__repr__
). You should call itrtn
or something else.I usually overwrite built-ins sometimes since in Python a lot of common words are build -ins.
Edited by Stephan Seitz
- Resolved by Stephan Seitz
- Resolved by Stephan Seitz
added 2 commits
added 1 commit
- 87f5cfdd - Replaced all format strings with f-strings in test cases
enabled an automatic merge when the pipeline for 76c3727b succeeds
mentioned in commit 82af488a