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) improve accounts state dump process #18

Merged
merged 7 commits into from
Jun 28, 2024
Merged

Conversation

cardosofede
Copy link
Contributor

No description provided.

@nikspz
Copy link
Contributor

nikspz commented Jun 25, 2024

Steps:

  1. Clone backend-apiPR18
  2. Manually build docker image with docker build -t hummingbot/backend-api:latest -f Dockerfile .
  3. clone https://github.com/hummingbot/deploy
  4. for setup.sh removed line "docker compose pull"
  5. bash setup.sh
  6. go to http://localhost:8501/Portfolio

When you start dashboard, issue:

Upon crosschecking with Ralph also happened on source


2024-06-25 06:14:00.860 Uncaught app exception
Traceback (most recent call last):
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 584, in _run_script
    exec(code, module.__dict__)
  File "/home/dashboard/frontend/pages/orchestration/portfolio/app.py", line 78, in <module>
    tokens_available = st.multiselect("Select Tokens", token_options, token_options)
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/runtime/metrics_util.py", line 397, in wrapped_func
    result = non_optional_func(*args, **kwargs)
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/elements/widgets/multiselect.py", line 249, in multiselect
    return self._multiselect(
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/elements/widgets/multiselect.py", line 292, in _multiselect
    indices = _check_and_convert_to_indices(opt, default)
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/elements/widgets/multiselect.py", line 92, in _check_and_convert_to_indices
    raise StreamlitAPIException(
streamlit.errors.StreamlitAPIException: Every Multiselect default value must exist in options
2024-06-25 06:20:46.486 Uncaught app exception
Traceback (most recent call last):
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 584, in _run_script
    exec(code, module.__dict__)
  File "/home/dashboard/frontend/pages/orchestration/portfolio/app.py", line 78, in <module>
    tokens_available = st.multiselect("Select Tokens", token_options, token_options)
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/runtime/metrics_util.py", line 397, in wrapped_func
    result = non_optional_func(*args, **kwargs)
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/elements/widgets/multiselect.py", line 249, in multiselect
    return self._multiselect(
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/elements/widgets/multiselect.py", line 292, in _multiselect
    indices = _check_and_convert_to_indices(opt, default)
  File "/opt/conda/envs/dashboard/lib/python3.10/site-packages/streamlit/elements/widgets/multiselect.py", line 92, in _check_and_convert_to_indices
    raise StreamlitAPIException(
streamlit.errors.StreamlitAPIException: Every Multiselect default value must exist in options

image

After accounts/ credentials added

image
image

@rapcmia
Copy link
Contributor

rapcmia commented Jun 25, 2024

Setup this PR with dashboard + hummingbot broker on source

  • Ongoing issue on backend-api BackendAPI - Missing data, conf/controller and conf/scripts  #17 not related to this PR
  • Managed to setup successfully on source ✅
  • Getting same behavior as Nikita reported for porfolio
    • By default with only master_account, getting the same error Uncaught app exception
    • Added test account, still same error
    • Added connectors to each accounts works ✅

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