Bug in the harmony module

[eidanch]

The following code:

>>> y = harmony.ChordSymbol('F')
>>> y.key = key.Key('C')
>>> y.romanNumeral

Produces the following TypeError:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\harmony.py", line 373, in fget
    self._roman = roman.romanNumeralFromChord(self)
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\roman.py", line 622, in romanNumeralFromChord
    chordObjSemiclosed = chordObj.semiClosedPosition(inPlace=False)
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\chord.py", line 2822, in semiClosedPosition
    c2 = self.closedPosition(forceOctave, inPlace, leaveRedundantPitches)
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\chord.py", line 920, in closedPosition
    returnObj = copy.deepcopy(self)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 174, in deepcopy
    y = copier(memo)
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\chord.py", line 277, in __deepcopy__
    new = note.NotRest.__deepcopy__(self, memo=memo)
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\note.py", line 599, in __deepcopy__
    new = GeneralNote.__deepcopy__(self, memo=memo)
  File "C:\Users\Eidan\Anaconda\lib\site-packages\music21\base.py", line 1679, in __deepcopy__
    newValue = copy.deepcopy(part, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 190, in deepcopy
    y = _reconstruct(x, rv, 1, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 334, in _reconstruct
    state = deepcopy(state, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 163, in deepcopy
    y = copier(x, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 257, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 190, in deepcopy
    y = _reconstruct(x, rv, 1, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 334, in _reconstruct
    state = deepcopy(state, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 163, in deepcopy
    y = copier(x, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 257, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 190, in deepcopy
    y = _reconstruct(x, rv, 1, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 334, in _reconstruct
    state = deepcopy(state, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 163, in deepcopy
    y = copier(x, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 257, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 163, in deepcopy
    y = copier(x, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 257, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 190, in deepcopy
    y = _reconstruct(x, rv, 1, memo)
  File "C:\Users\Eidan\Anaconda\lib\copy.py", line 329, in _reconstruct
    y = callable(*args)
  File "C:\Users\Eidan\Anaconda\lib\copy_reg.py", line 93, in __newobj__
    return cls.__new__(cls, *args)
TypeError: object.__new__(NotImplementedType) is not safe, use NotImplementedType.__new__()

[mscuthbert]

Still needs a bit more testing, but a patch is in. change

    @key.setter
    def key(self, keyOrScale):
        if common.isStr(keyOrScale):
            self._key = key.Key(keyOrScale)
        else:
            self._key = key

to

    @key.setter
    def key(self, keyOrScale):
        if common.isStr(keyOrScale):
            self._key = key.Key(keyOrScale)
        else:
            self._key = keyOrScale
            self._roman = None

or upgrade to latest SVN.

Apparently the error comes from trying to deepcopy the key, MODULE object.