Skip to content

Feature AlphaSwarmTool & docs#75

Merged
Winston-503 merged 43 commits intomainfrom
feature-standardize-tools
Feb 20, 2025
Merged

Feature AlphaSwarmTool & docs#75
Winston-503 merged 43 commits intomainfrom
feature-standardize-tools

Conversation

@Winston-503
Copy link
Contributor

@Winston-503 Winston-503 commented Feb 13, 2025

  • Introduce AlphaSwarmTool
  • Rename tool implementation files to not contain _tool suffix
  • Rename tool classes to not contain Tool suffix
  • Rename tools to reflect the action taken, e.g. from GenericStrategyAnalysisTool to AnalyzeTradingStrategy
  • Proper imports in __init__(s)
  • Add docs for tool and services contributions

@Winston-503 Winston-503 changed the title Feature standardize tools Feature AlphaSwarmTool Feb 14, 2025
# Conflicts:
#	alphaswarm/services/exchanges/base.py
#	alphaswarm/services/exchanges/jupiter/jupiter.py
#	tests/integration/tools/exchanges/test_execute_token_swap_tool.py
def adapt(cls, alphaswarm_tool: AlphaSwarmToolBase) -> Tool:
tool = Tool()

tool.name = alphaswarm_tool.name
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to remove the always possible Tool suffix?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I think it's more important to be consistent with the class name

Winston-503 and others added 16 commits February 20, 2025 08:37
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
Co-authored-by: Arnaud Flament <17051690+aflament@users.noreply.github.com>
# Conflicts:
#	alphaswarm/tools/telegram/send_telegram_notification.py
#	examples/interaction/terminal.py
@Winston-503 Winston-503 merged commit f92d3fd into main Feb 20, 2025
2 checks passed
@Winston-503 Winston-503 deleted the feature-standardize-tools branch February 20, 2025 14:30
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