bbgo icon indicating copy to clipboard operation
bbgo copied to clipboard

FEATURE: [indicator] adding a bunch of new v2 indicators

Open go-dockly opened this issue 2 years ago • 27 comments
trafficstars

go-dockly avatar Oct 26 '23 03:10 go-dockly

Hi @go-dockly,

This is KarmaBot, and we reward your contributions with tokens sent directly to your wallet to support development.

This pull request may get 4869 BBG.

To receive BBG tokens, please provide your Polygon (can be Ethereum) address as an issue comment in this pull request, following this format:

polygon:0xAb5801a7D398351b8bE11C439e05C5B3259aeC9B

Once this pull request is merged, your BBG tokens will be transferred to your wallet.

-- If you're interested in our project, feel free to join our Telegram group https://t.me/bbgo_intl

Best, KarmaBot

bbgokarma-bot avatar Oct 26 '23 03:10 bbgokarma-bot

Re-estimated karma: this pull request may get 9820 BBG

bbgokarma-bot avatar Oct 26 '23 22:10 bbgokarma-bot

Re-estimated karma: this pull request may get 9931 BBG

bbgokarma-bot avatar Oct 27 '23 00:10 bbgokarma-bot

Re-estimated karma: this pull request may get 10340 BBG

bbgokarma-bot avatar Oct 27 '23 20:10 bbgokarma-bot

Re-estimated karma: this pull request may get 10633 BBG

bbgokarma-bot avatar Oct 27 '23 23:10 bbgokarma-bot

Re-estimated karma: this pull request may get 11162 BBG

bbgokarma-bot avatar Oct 29 '23 21:10 bbgokarma-bot

this PR is getting too big to merge, would you mind to split them and maybe we can merge some changes, like the csv source package, and the kline csv readers, or maybe the extended slice methods.

c9s avatar Oct 30 '23 00:10 c9s

Re-estimated karma: this pull request may get 11026 BBG

bbgokarma-bot avatar Oct 30 '23 01:10 bbgokarma-bot

Re-estimated karma: this pull request may get 11031 BBG

bbgokarma-bot avatar Oct 30 '23 01:10 bbgokarma-bot

Re-estimated karma: this pull request may get 11195 BBG

bbgokarma-bot avatar Oct 30 '23 18:10 bbgokarma-bot

Re-estimated karma: this pull request may get 11654 BBG

bbgokarma-bot avatar Oct 31 '23 02:10 bbgokarma-bot

Re-estimated karma: this pull request may get 11600 BBG

bbgokarma-bot avatar Oct 31 '23 16:10 bbgokarma-bot

Re-estimated karma: this pull request may get 11650 BBG

bbgokarma-bot avatar Oct 31 '23 22:10 bbgokarma-bot

There are too many files, like 140+ files that I could not review all files in one round, will continue the review later.

Since you're the first-time contributor, it would be nice if you can submit a pull request with a small change, so that the CI could be executed for your later PRs.

c9s avatar Nov 02 '23 04:11 c9s

Or, maybe you can pull out the csvsource changes to one PR, I've reviewed these csvsource file changes.

c9s avatar Nov 02 '23 04:11 c9s

Re-estimated karma: this pull request may get 8952 BBG

bbgokarma-bot avatar Nov 02 '23 16:11 bbgokarma-bot

Re-estimated karma: this pull request may get 8224 BBG

bbgokarma-bot avatar Nov 02 '23 16:11 bbgokarma-bot

Re-estimated karma: this pull request may get 8229 BBG

bbgokarma-bot avatar Nov 02 '23 16:11 bbgokarma-bot

This build is failing due to the go.mod specified compiler v1.18. Can we bump to latest go please?

go-dockly avatar Nov 03 '23 14:11 go-dockly

let’s go with go 1.20

go-dockly @.***>於 2023年11月3日 週五,22:54寫道:

This build is failing due to the go.mod specified compiler v1.18. Can we bump to latest go please?

— Reply to this email directly, view it on GitHub https://github.com/c9s/bbgo/pull/1366#issuecomment-1792589170, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMNTSVHFERSR2Y7HOMUELYCUAQVAVCNFSM6AAAAAA6QL6GRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJSGU4DSMJXGA . You are receiving this because you commented.Message ID: @.***>

c9s avatar Nov 03 '23 14:11 c9s

Re-estimated karma: this pull request may get 8364 BBG

bbgokarma-bot avatar Nov 06 '23 22:11 bbgokarma-bot

Re-estimated karma: this pull request may get 8369 BBG

bbgokarma-bot avatar Nov 06 '23 22:11 bbgokarma-bot

Re-estimated karma: this pull request may get 8379 BBG

bbgokarma-bot avatar Nov 06 '23 22:11 bbgokarma-bot

Re-estimated karma: this pull request may get 8399 BBG

bbgokarma-bot avatar Nov 06 '23 22:11 bbgokarma-bot

I think this PR is good to merge now

go-dockly avatar Nov 06 '23 22:11 go-dockly

@go-dockly could you please rebase the commits? make it no merge-commits in your PR, thanks

c9s avatar Dec 08 '23 07:12 c9s

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.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jan 24 '24 07:01 CLAassistant