You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The recent attrs release deprecates repr_ns. We use this feature to facilitate separating the fwdpy11 namespace across multiple Python files. Such separation is NOT really supported by the language, meaning that what we have done is a hack.
#1330 pinned attrs to avoid the new release in order to get CI to work.
But we can't live with a ceiling pin like this, so we have to address the deprecation.
Experiments in #1331 show that the only think that breaks by simply removing all use of repr_ns is a little-used feature: recreating Python objects from metadata in tskit tree sequences. There is no effect on recreating mutations, etc.. Rather, recreating thing like ModelParams, which contains other fwdpy11 objects. The issue is that the objects will no longer contain their namespace in their __repr__, resulting in NameError. Maintaining current behavior seems difficult in the best case.
The text was updated successfully, but these errors were encountered:
The recent attrs release deprecates
repr_ns
. We use this feature to facilitate separating the fwdpy11 namespace across multiple Python files. Such separation is NOT really supported by the language, meaning that what we have done is a hack.#1330 pinned attrs to avoid the new release in order to get CI to work.
But we can't live with a ceiling pin like this, so we have to address the deprecation.
Experiments in #1331 show that the only think that breaks by simply removing all use of
repr_ns
is a little-used feature: recreating Python objects from metadata in tskit tree sequences. There is no effect on recreating mutations, etc.. Rather, recreating thing like ModelParams, which contains other fwdpy11 objects. The issue is that the objects will no longer contain their namespace in their__repr__
, resulting inNameError
. Maintaining current behavior seems difficult in the best case.The text was updated successfully, but these errors were encountered: