Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python 2.6 does not have WeakSet #66

Closed
jjaakola opened this issue Nov 18, 2014 · 2 comments
Closed

Python 2.6 does not have WeakSet #66

jjaakola opened this issue Nov 18, 2014 · 2 comments

Comments

@jjaakola
Copy link

Python Package Index states 2.6 as supported in Music21 1.9.3, https://pypi.python.org/pypi/music21.
The Python class weakref.WeakSet is used in streams.timespans: https://github.com/cuthbertLab/music21/search?utf8=%E2%9C%93&q=WeakSet

WeakSet is introduced in Python 2.7.

Easy to test with Python 2.6:

from music21 import *
stream.timespans.makeExampleScore()

This will result in exception: AttributeError: 'module' object has no attribute 'WeakSet'

@mscuthbert
Copy link
Member

I will need to update the Python Package Index — unfortunately the information is out of date. Music21 since, I believe 1.8 or 1.9 requires Python 2.7. The only thing that was added in 1.9 is the functionality for speed that WeakSet gives so a shim to continue to support 2.6 will not add new features.

Best,
Myke

On Nov 18, 2014, at 12:05, jjaakola <notifications@github.commailto:notifications@github.com> wrote:

Python Package Index states 2.6 as supported in Music21 1.9.3, https://pypi.python.org/pypi/music21.
The Python class weakref.WeakSet is used in streams.timespans: https://github.com/cuthbertLab/music21/search?utf8=%E2%9C%93&q=WeakSet

WeakSet is introduced in Python 2.7.

Easy to test with Python 2.6:

from music21 import *
stream.timespans.makeExampleScore()

This will result in exception: AttributeError: 'module' object has no attribute 'WeakSet'


Reply to this email directly or view it on GitHubhttps://github.com//issues/66.

@mscuthbert
Copy link
Member

Updated PPI to show that 2.7 is now the required version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants