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(690) list index out of range in crew.py #742

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mangalprada
Copy link

Handle empty metrics list to avoid IndexError in crew.py mentioned in #690

  • Added a condition to check if the metrics list is empty before accessing its elements.
  • This prevents IndexError by setting self.usage_metrics to an empty dictionary if metrics is empty.

fix(logger): Resolve NameError by importing datetime module

  • Added missing import for datetime module to resolve NameError.
  • Ensured datetime is correctly used in the log method for timestamp generation.

@SimasJan
Copy link

Ended up here due to the same issue as in #690 .
Can this be closed so latest version has this fixed?
Currently implemented the fix manually 😅 Thanks

@SimasJan
Copy link

Found a duplicate of this issue #262
@joaomdmoura when would this fix be merged? If you are looking for extra volunteers to support on maintaining the package, would be happy to contribute :)

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.

None yet

2 participants