这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@cardosofede
Copy link
Contributor

Changes:

  • added isort, black, flake and pre-commit
  • format all the files
  • remove unused modules
  • remove quants lab

@rapcmia rapcmia requested review from nikspz and rapcmia July 17, 2024 02:32
@rapcmia
Copy link
Contributor

rapcmia commented Jul 17, 2024

PR update:

  • Backend and dashboard source + broker compose file, no go and created bug ticket 25
  • Dashboard source + backend and broker compose file ok used on testing this PR
    image

Added isort, black, flake8 and pre-commit

  • Remove and recreate env for dashboard ok
  • Not sure how we can check black ?
    image

initial tests:

  • Test bot orchestration
    • Accounts with credentials (CRUD) and portfolio ok
    • Instances and random config-generator ok
    • Ran test with random config-generator and switch to diff config ok

Todo:

  • Run tests on all config-generator
  • Run backtests on pmm controllers
  • Deploy and observe instance

@nikspz
Copy link
Contributor

nikspz commented Jul 17, 2024

Test perfromed:

  • Clone dashboardPR160 + backendPR24
  • Removed dashboard and backend-api environment
  • set up env for dashboardPR160 + backendPR24 again
  • start dashboard and backend-api
  • Run backtesting successfully on PMM simple, PMM Dynamic, D-man maker v2, Bollinger bands, MACD_BB

Pending:
Instance test with hummingbot/hummingbot#7121

Copy link
Contributor

@rapcmia rapcmia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Setup this PR with backend PR24, broker from deploy docker compose and deploy successfully hummingbot [PR7121](https://github.com/hummingbot/hummingbot/pull/7121) image
  • Added account and credentials
    • Create, update and remove random connectors
    • Check portfolio matched on exchange current assets
  • Ran backtests few backtesting using kucoin and binance on PMM_simple and PMM_dynamic
  • Ran random configs and switched to other controllers to check possible mixed setup, all ok
  • Ran random configs and deploy using kucoin and binance_perpetual as connector ok

@nikspz nikspz merged commit 7f0df00 into development Jul 18, 2024
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.

4 participants