-
-
Notifications
You must be signed in to change notification settings - Fork 194
Feat/bot father #31
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/bot father #31
Conversation
|
When I created a bot, I ran into an error b/c it couldn't find a Afterwards, I was able to use the broker to create an instance. However, when I attached to it, it didn't print a I see this error in the Dashboard: I suspect my broker isn't connected to the bot |
|
@fengtality I think you need to run |
|
After the last commit, bot instances should now auto-connect to Broker, allowing you to test the strategy start/stop controls. To test a script, copy simple_pmm_example.py into the You may need to stop and start the instance, the bot can only access script available at startup. Afterwards, you should be able to start, get status, and stop the script from the dashboard! |
|
Latest commit 1658290: Test performed:
redundant .yml file in strategy start:
failed to start the .yml file from dashboard |
|
to previous bug: Steps: |
hbotrc/__init__.py
Outdated
| @@ -0,0 +1,8 @@ | |||
| __version__ = "0.1.0" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer to install it as a package in environment_conda.yml instead of duplicating the code.
name: dashboard
channels:
- conda-forge
dependencies:
- python=3.9
- sqlalchemy
- pip
- pip:
- ccxt
- streamlit
- watchdog
- plotly
- pycoingecko
- glom
- defillama
- statsmodels
- pandas_ta
- pyyaml
- commlib-py
- git@github.com:hummingbot/hbot-remote-client-py.git
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes I agree, but in this initial draft I made some quick changes to the code and I was testing how to run async tasks in streamlit but definitely we need to use the same repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me know please how I can help, I do want to help with something, but I don't want us to double code it, or we create code conflicts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perfect! Tomorrow we have the first live session. I think that @fengtality is going to create some tasks around how we can start working together on this.
from my side I will clean up this branch and probably we can merge it so people can start using it and then we can split our work in different pages
PR to manage multiple bots from the dashboard 🚀