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

fix(typings): Ease restrictions. #388

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

miladtsx
Copy link

No description provided.

@miladtsx
Copy link
Author

Hi @Adamantios @dagacha ,
I encountered issues executing the scripts on my local system, which led to these adjustments.

Example error:

cd trader; poetry run python ../trades.py --creator 0x247F466F6deaC348F694725F5abd0aC6309125F5; cd ..
Traceback (most recent call last):
  File "../trades.py", line 36, in <module>
    from scripts.mech_events import get_mech_requests
  File "/home/ilad/labs/olas/trader-quickstart/scripts/mech_events.py", line 196, in <module>
    sender: str, event_cls: type[MechBaseEvent]
TypeError: 'type' object is not subscriptable

My System details:

  • OS: Linux-Fedora
  • Python 3.8.18
  • Poetry: 1.8.4
  • Poetry-Core Version: 1.9.1

Let me know if you need further context or clarification.

Copy link
Collaborator

@Adamantios Adamantios left a comment

Choose a reason for hiding this comment

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

Please rebase on develop as there are unsynced changes.

report.py Outdated Show resolved Hide resolved
report.py Outdated Show resolved Hide resolved
trades.py Outdated Show resolved Hide resolved
miladtsx and others added 3 commits November 22, 2024 18:13
Co-authored-by: Adamantios Zaras <[email protected]>
Co-authored-by: Adamantios Zaras <[email protected]>
Co-authored-by: Adamantios Zaras <[email protected]>
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