Skip to content

Update algo.py#8

Open
sgtg-us wants to merge 1 commit intoalpacahq:masterfrom
sgtg-us:master
Open

Update algo.py#8
sgtg-us wants to merge 1 commit intoalpacahq:masterfrom
sgtg-us:master

Conversation

@sgtg-us
Copy link
Copy Markdown

@sgtg-us sgtg-us commented Jan 7, 2020

Changed macd import to Class Based due to changes in ta module. https://technical-analysis-library-in-python.readthedocs.io/en/latest/ta.html#ta.trend.MACD

Copy link
Copy Markdown

@wilbobraggins wilbobraggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not just:

from ta.trend import macd

@nickvertucci
Copy link
Copy Markdown

@sgtg-us do you have a working verison of this algo?

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sramekmichal
Copy link
Copy Markdown

sramekmichal commented Nov 1, 2023

Traceback (most recent call last):
  File "/home/<username>/alg0.py", line 369, in <module>
    if __name__ == __main__:
NameError: name '__main__' is not defined

Do not know, where is the problem now.

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

Successfully merging this pull request may close these issues.

4 participants