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

Failed Test PR #11 - Unhandled error in background task #13

Closed
mrhzysbl opened this issue Apr 3, 2023 · 2 comments
Closed

Failed Test PR #11 - Unhandled error in background task #13

mrhzysbl opened this issue Apr 3, 2023 · 2 comments

Comments

@mrhzysbl
Copy link

mrhzysbl commented Apr 3, 2023

Summary and Screenshots

After starting the strategy, an error message is shown from the log pane.

Screen Shot 2023-04-03 at 2 35 12 PM

2023-04-03 14:30:15,987 - 12769 - hummingbot.core.utils.async_utils - ERROR - Unhandled error in background task: [<class 'decimal.InvalidOperation'>]
Traceback (most recent call last):
  File "/Users/admin/pr12-client/hummingbot/core/utils/async_utils.py", line 9, in safe_wrapper
    return await c
  File "/Users/admin/pr12-client/hummingbot/strategy/amm_arb/amm_arb.py", line 213, in main
    profitable_arb_proposals: List[ArbProposal] = [
  File "/Users/admin/pr12-client/hummingbot/strategy/amm_arb/amm_arb.py", line 215, in <listcomp>
    if t.profit_pct(
decimal.InvalidOperation: [<class 'decimal.InvalidOperation'>] 

Steps To Reproduce

  1. Clone and install the gateway repo and dev branch of Hummingbot OSS
  2. Connect the client to gateway
  3. Create AMM_Arbritage strategy
  4. Start the bot

Expected Results

The bot will start without any errors

Test Result Comparison

N/A, already in dev branch

Attachments

ZigZag.zip

@mrhzysbl mrhzysbl linked a pull request Apr 3, 2023 that will close this issue
2 tasks
@mrhzysbl mrhzysbl transferred this issue from CoinAlpha/hummingbot Apr 3, 2023
@mrhzysbl mrhzysbl removed a link to a pull request Apr 4, 2023
2 tasks
@mrhzysbl mrhzysbl linked a pull request Apr 4, 2023 that will close this issue
2 tasks
@vic-en
Copy link

vic-en commented Apr 6, 2023

@mrhzysbl I fixed this. pls confirm.

@mrhzysbl
Copy link
Author

mrhzysbl commented Apr 7, 2023

Confirmed working after commit d518c63

@mrhzysbl mrhzysbl closed this as completed Apr 7, 2023
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 a pull request may close this issue.

2 participants