search

GregorCH / ipet

Interactive Performance Evaluation Tools for Optimization Software
MIT License
26 stars 6 forks source link

update deps #88

Closed spoorendonk closed 5 years ago

spoorendonk commented 5 years ago

Updating to support python 3.7. Command line only. PyQt4 fails.

GregorCH commented 5 years ago

What a coincidence. Just today, I have had a discussion about the old dependencies. Have you tested it?

spoorendonk commented 5 years ago

Tests are ok except for the pandas warning.

$ python -m unittest test
/home/simon/flowty/playground/ipet/ipet/Experiment.py:17: FutureWarning: The Panel class is removed from pandas. Accessing it from the top-level namespace will also be removed in the next version
  from pandas import Panel
..................................
----------------------------------------------------------------------
Ran 34 tests in 46.376s

OK

As mentioned I did not test the gui part because I could not get PyQt4 to work and porting to PyQt5 was not trivial because of SIGNAL() deprecation.

GregorCH commented 5 years ago

Here is another warning that I was told about today, this time with pandas 0.24 (this PR updates to 0.25). I am happy to merge this as all tests run through.

Experiment.py:246: FutureWarning: Sorting because non-concatenation axis is not aligned. A future version
of pandas will change to not sort by default.

To accept the future behavior, pass 'sort=False'.

To retain the current behavior and silence the warning, pass 'sort=True'.

  return pd.concat(datalist).infer_objects()
GregorCH commented 5 years ago

FYI @fschloesser

GregorCH commented 5 years ago

The panel stuff is no longer used, anyway. It was used to build a hierarchical data frame, which is no longer necessary because of the index logic that we introduced later.