Skip to content

Conversation

Youbadawy
Copy link
Contributor

Dear all,

Hope this pull requests finds you all well and in good health,

Sorry for the delay, as requested this is a smaller pull request, with pep8 styling and documentation for the config items,

Goal of PR,

the primary aim for these modules is to facilitate the extraction of data, information, agents, and what not from a unified, standardized location in the repository ( ./user_data).

also I believe we should change the location of the config.json file that was put inside /notebooks to a central config folder in the parent directory. where we will put an example file to retrieve if any user need.

Whats next?

next we have the data fetching for crypto using the ccxt module.

@XiaoYangLiu-FinRL @BruceYanghy @spencerR1992

hope that meets the requirements...

@zhumingpassional
Copy link
Collaborator

thanks for your valuable codes.

could you pls separate this PR to several PRs?
In software engineering, we hope each PR solves only one problem.

In addition, this branch has conflicts with master. could you pls revise it?

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.

2 participants