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

Feat/clean up repo #160

Merged
merged 23 commits into from
Jul 18, 2024
Merged

Feat/clean up repo #160

merged 23 commits into from
Jul 18, 2024

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 rapcmia and nikspz 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.

3 participants