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

No support for Candles endpoint #5

Open
markwkjr opened this issue Oct 18, 2017 · 10 comments
Open

No support for Candles endpoint #5

markwkjr opened this issue Oct 18, 2017 · 10 comments

Comments

@markwkjr
Copy link
Contributor

markwkjr commented Oct 18, 2017

No support candles endpoint

This endpoint returns Array rather than JSON so requiring new methods. I am working on code to support this endpoint.

@sefbkn
Copy link
Owner

sefbkn commented Oct 19, 2017

@markwkjr That would be fantastic.

@sefbkn sefbkn closed this as completed Oct 19, 2017
@sefbkn sefbkn reopened this Oct 19, 2017
@UniversalAE
Copy link
Contributor

Is there any progress on support for the candles end point. What's the ETA?

@UniversalAE
Copy link
Contributor

sefbkn maekwkjr I have a crude Candles endpoint working if you want it. Let me know here and I'll try to figure out how to post it.

@markwkjr
Copy link
Contributor Author

That'd be great @UniversalAE
Just fork, commit, and submit a Pull Request.
May want to check out a blog, such as: https://blog.scottlowe.org/2015/01/27/using-fork-branch-git-workflow/

@UniversalAE
Copy link
Contributor

UniversalAE commented Nov 20, 2017

yes, I am aware of the error and have corrected this. i'm glad you are interested. I will investigate posting. Right now I have the return type as JArray. I'm just about to do the necessary parsing. I will fork when this is complete unless there is a more private way to release to you. I think my changes should go in the main body of work. I'm not interested in attribution. I have tried to add the endpoint following your style of coding. There is an additional type Candles.cs and slight changes to ProductClient.cs and OrderClient.cs. I've also overloaded the PlaceOrderAsync interface to add post_only as an option.

@UniversalAE
Copy link
Contributor

I'm done. Works great. Investigating the Pull request. Look in ProductClient.cs for recommended usage example.

@UniversalAE
Copy link
Contributor

QED

@UniversalAE
Copy link
Contributor

Dammit! I forgot one last cleanup item. The Candle granularity can be down to one second. I was testing in 1 minute intervals and had obscured the seconds in string formatting ISO 8601.
yyyy-MM-ddTHH:mm:00.00000Z should be changed to yyyy-MM-ddTHH:mm:ss.00000Z. The change should be made in ProductClient.cs .

@UniversalAE
Copy link
Contributor

One more Dammit! I had to move the line Candle c = new Candle(); inside the while loop. It was working last night the way I posted it. This morning the 200 candle list all had the same candle data for each record. Moving the class initializer inside the loop corrects the problem but I do not comprehend why it fixes it. Play with it on your end to see if you get the same results.

@corcoranp
Copy link

In the ProductClient.cs, I noticed the candle data was returning the same values for all items. I updated the code to new up the candle on each iteration and that fixed it. I also wanted the latest values in the front of the array so I changed the loop.

Final code looks like:

for (int i = 0; i <= x.Count - 1; i++){
c = new Candle(); //added by PMC
c.time = (long)x[i][0];
c.low = (decimal)x[i][1];
c.high = (decimal)x[i][2];
c.open = (decimal)x[i][3];
c.close = (decimal)x[i][4];
c.volume = (decimal)x[i][5];
cl.Add(c);
}

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

4 participants