-
Notifications
You must be signed in to change notification settings - Fork 2
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
Refactor DeployableTraderAgent to 1. predict a probability for a market then 2. place a bet based on its BettingStrategy
#338
Comments
@gabrielfior says "don't change the default for all agents... the dune dashboard will get messed up. Try on one or a few agents first, see how it changes accuracy(?) and ROI" |
Re-opening, as #352 didn't actually change the default |
@evangriffiths do you mind closing this ticket in favor of #365? |
BettingStrategy
Sure. Have renamed to make it clearer what we're closing as solved 😄 |
May need a refactor of
DeployableTraderAgent.process_bets
(seeprediction-market-agent-tooling/prediction_market_agent_tooling/deploy/agent.py
Line 281 in 69ec60e
Currently it does:
The abstraction is a bit off here if we want to use
get_kelly_bet
here, as we want to calculate the direction and amount together, so it would look like:The text was updated successfully, but these errors were encountered: