Musical metaphor

[ksunden]

No description provided.

[lgtm-com]

This pull request introduces 3 alerts when merging c610187 into bdb5562 - view on LGTM.com

new alerts:

• 2 for Module imports itself
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 4 alerts when merging 1e47d5f into bdb5562 - view on LGTM.com

new alerts:

• 2 for Module imports itself
• 1 for First parameter of a method is not named 'self'
• 1 for 'import *' may pollute namespace

[ghost]

DeepCode's analysis on #c98e00 found:

• ❌ 1 critical issue, ⚠️ 2 warnings, ℹ️ 8 minor issues. 👇

Top issues

Description	Example fixes
Use binary mode in open (current mode is w) to avoid encoding-related issues for written file, on Windows or with Python 3. Occurrences: _store.py:133	🔧 Example fixes
Defining only __eq__ but not __ne__ will result in a Python2 error if objects are compared with inequality. Occurrences: _motor.py:11 _instrument.py:33 _tune.py:53	🔧 Example fixes
Missing close for open, add close or use a with block. Occurrences: setup.py:12	🔧 Example fixes

👉 View analysis in DeepCode’s Dashboard | Configure the bot

[lgtm-com]

This pull request introduces 2 alerts when merging 6c0febb into bdb5562 - view on LGTM.com

new alerts:

• 1 for Comparison of identical values
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 3 alerts when merging 45df213 into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused import
• 1 for Comparison of identical values
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 2 alerts when merging 635507b into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused import
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 2 alerts when merging 8a87992 into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused import
• 1 for 'import *' may pollute namespace

[lgtm-com]

This pull request introduces 3 alerts when merging 7bac7b7 into bdb5562 - view on LGTM.com

new alerts:

• 1 for Wrong name for an argument in a call
• 1 for Unused import
• 1 for Explicit export is not defined

[lgtm-com]

This pull request introduces 2 alerts when merging bac5755 into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused import
• 1 for Explicit export is not defined

[lgtm-com]

This pull request introduces 1 alert when merging 44a0cca into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused import

[lgtm-com]

This pull request introduces 1 alert when merging 2797fd2 into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused local variable

[lgtm-com]

This pull request introduces 1 alert when merging 2d17ebf into bdb5562 - view on LGTM.com

new alerts:

• 1 for Unused local variable