-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
date_time_parser #25763
date_time_parser #25763
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
Hello, would you mind adding a test for the new functionality? Thanks. |
except Exception as e: | ||
# If any other unexpected exception type is raised | ||
if expected is None: | ||
assert True # Correctly identified that input was invalid | ||
else: | ||
pytest.fail(f"Unhandled exception {e} for input {x}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what other exceptions are we expecting here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could be redundant, but was just worried if something like lark error might go unnoticed silently.
Thank you for contributing to LangChain!
PR title: "langchain: Chains: query_constructor: add date time parser"
PR message:
Add tests and docs: If you're adding a new integration, please include
docs/docs/integrations
directory.Lint and test: Run
make format
,make lint
andmake test
from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/Additional guidelines:
If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.