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

Python implementation of OHLCV aggregation method #751

Closed
wants to merge 47 commits into from

Conversation

samholt
Copy link
Contributor

@samholt samholt commented Dec 5, 2017

Direct port from: https://github.com/ccxt/ccxt/pull/695/files

( Also is the Websockets WIP (Async) in python, however feel free to exclude this. )

@kroitor
Copy link
Member

kroitor commented Dec 5, 2017

Tons of thanks!

@kroitor kroitor self-assigned this Dec 5, 2017
@kroitor kroitor mentioned this pull request Dec 6, 2017
@kroitor
Copy link
Member

kroitor commented Dec 10, 2017

@bitsol merging it now, in progress.

samholt and others added 2 commits December 14, 2017 19:16
that proxy selected, as was noticing proxy changing errors when using
multiple proxies, and them being set in fetch2.
@samholt samholt closed this Dec 15, 2017
@samholt samholt reopened this Dec 15, 2017
@fuzzyTew
Copy link

It would be great to have the websocket stuff on a separate branch & PR, for others to look at and consider using and contributing to.

@fuzzyTew
Copy link

I believe I've separated out just the websocket work at https://github.com/fuzzyTew/ccxt/tree/websockets . Haven't reviewed it to see how much more work is needed. It at least needs to be ported to javascript.

@fuzzyTew
Copy link

It looks like samholt copied the python base exchange class, deleted the rest-api methods, and added in websocket-api methods.

I'm thinking it might be more DRY to factor the rest/websocket-api functions out of the exchange base class and put them into differently named utility classes. Then exchange implementations can include one or more utility classes if they need, or perhaps even query to see if different utility classes exist on their platform to determine what functionality to advertise. This would make things like generalized tcp streams or platform-specific pusher implementations reasonable in the future.

I found some old chat logs discussing this in a very general way ... is there some chat where ccxt development is discussed?

@kroitor
Copy link
Member

kroitor commented Dec 26, 2017

@fuzzyTew, sure:

Gitter

@kroitor
Copy link
Member

kroitor commented Feb 26, 2018

Merged OHLCV functions in #894

@kroitor kroitor mentioned this pull request Mar 9, 2018
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.

None yet

3 participants