blankly-finance / blankly

🚀 💸 Easily build, backtest and deploy your algo in just a few lines of code. Trade stocks, cryptos, and forex across exchanges w/ one package.
https://package.blankly.finance
GNU Lesser General Public License v3.0
2.03k stars 261 forks source link

Added pandas dataframe to PriceReader data input. #216

Closed frostja closed 1 year ago

frostja commented 1 year ago

Added the ability to pass a dataframe instead of a csv or json file to PriceReader function.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Jack Frost seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Jack Frost seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

sonarcloud[bot] commented 1 year ago

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
0.0% 0.0% Duplication

EmersonDove commented 1 year ago

This is awesome, if you can do the CLA real quick I can merge it in.

frostja commented 1 year ago

When I click the CLA then I get: “You have agreed to the CLA for blankly-finance/blankly” but then the commit page does not look to me like it is done. If it's not done let me know.

EmersonDove commented 1 year ago

Oh interesting it looks like CLA assistant isn't updating

EmersonDove commented 1 year ago

Merged & released in the mainline package, thanks for this pr.