search

markovmodel / msmtools

Tools for estimating and analyzing Markov state models
GNU Lesser General Public License v3.0
40 stars 26 forks source link

rename msm.io to avoid shadowing python package io #12

Closed franknoe closed 9 years ago

franknoe commented 9 years ago

(moved from pyEMMA) since this breaks the api, we should do this in 1.3 and deprecate it somehow (I currently do not know what is the best practice to deprecate a package name).

But actually the issue of shadowing is not a big deal, since it only affects imports one level above "io". So in principle we can not import the builtin.io at level of the msm package. In my opinion this only creates work and does not solve any real problem.

marscher commented 9 years ago

But actually the issue of shadowing is not a big deal, since it only affects imports one level above "io". So in principle we can not import the builtin.io at level of the msm package. In my opinion this only creates work and does not solve any real problem. Agreed. We should leave it as it is by now.

franknoe commented 9 years ago

ok, then I close this.