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

[Bug] stocks/ta/ema --export EMA values are not exported, only "Adj Close" and "date". #5213

Closed
deeleeramone opened this issue Jul 11, 2023 · 5 comments · Fixed by #5214
Closed
Labels
bug Fix bug

Comments

@deeleeramone
Copy link
Contributor

To recreate,

/stocks/ta/ $ ema --length 10,20,30 --export csv

Screenshot 2023-07-11 at 12 57 23 PM

@deeleeramone
Copy link
Contributor Author

Same issue, /forex/ta/ema. Also the filename gets "stocks" inserted into it despite being from the forex path.

Screenshot 2023-07-14 at 10 55 30 AM

@deeleeramone deeleeramone reopened this Jul 14, 2023
@deeleeramone
Copy link
Contributor Author

Ditto that on everywhere else, stocks/crypto. The headers are created, but the data is missing.

@Ltwo3five
Copy link
Contributor

Hi is anyone working on this? can I work on it?

@deeleeramone
Copy link
Contributor Author

@tehcoderer is this patched somewhere in another PR?

@deeleeramone
Copy link
Contributor Author

Following up on this, @tehcoderer, if you do: /stocks/load aapl/ta/ema 50,150 --export csv

You get an exported file where the 50 day EMA is truncated to begin at row 151 instead of the full column of data beginning at row 51.

Screenshot 2023-08-22 at 12 37 46 PM

@jmaslek jmaslek closed this as completed May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants