MurpheyLab / trep

Trep: Mechanical Simulation and Optimal Control Software
https://murpheylab.github.io/trep/
GNU General Public License v3.0
18 stars 11 forks source link

scipy version incompatibility with trep.system.load_trajectory and trep.system.save_trajectory #14

Closed adwilson10 closed 10 years ago

adwilson10 commented 10 years ago

From schultzj...@gmail.com on October 23, 2012 10:54:42

Scipy changed the handling of 1D arrays in the scipy.io.savemat method. In older versions it defaulted to columns, and in newer versions it changed to rows. The 'savemat' function also What steps will reproduce the problem? 1. Save a system trajectory with an older version of scipy.

  1. Try and load the trajectory with a newer version of scipy.
  2. The issue is also reproducible in reverse. What is the expected output? What do you see instead? 1. It should load the trajectory and return a dict. Instead an error is thrown.

We need to modify the load and save trajectory methods to be agnostic as to whether the 1D arrays are rows or columns.

Original issue: http://code.google.com/p/trep/issues/detail?id=14

adwilson10 commented 10 years ago

From schultzj...@gmail.com on October 23, 2012 08:55:26

Labels: -Priority-Medium Priority-Low

adwilson10 commented 10 years ago

From schultzj...@gmail.com on October 23, 2012 12:26:07

Fixed in rev f57f7d55e8b3

Status: Fixed