Return none as fallback Helicone

[waynehamadi]

Background

Changes

PR Quality Checklist

• I have run the following commands against my code to ensure it passes our linters:

black . --exclude test.py
isort .
mypy .
autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring agbenchmark"
agbenchmark/start_benchmark.py

[github-actions]

PR Analysis

• 🎯 Main theme: Change error handling in get_data_from_helicone function
• 📌 Type of PR: Refactoring
• 🧪 Relevant tests added: No
• ✨ Focused PR: Yes, because the title, description, and changes are all related to the same function and its error handling.
• 🔒 Security concerns: No, because the changes are related to error handling and do not introduce any new functionality that could lead to security issues.

PR Feedback

• 💡 General PR suggestions: The PR is generally well-structured and focused. However, it would be beneficial to add tests to ensure that the new error handling works as expected. Also, consider logging errors instead of just printing them, to provide more robust error tracking.

How to use

To invoke the PR-Agent, add a comment using one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe: Modify the PR title and description based on the contents of the PR.
/improve: Suggest improvements to the code in the PR.
/ask <QUESTION>: Pose a question about the PR.

To edit any configuration parameter from 'configuration.toml', just add -config_path=. For example: '/review --pr_reviewer.extra_instructions="focus on the file: ..."'