-
Notifications
You must be signed in to change notification settings - Fork 160
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
extracting model response #73
Conversation
@marshmellow77 Thanks for contributing. Can you please update/run integration tests? |
Integration test ran successfully: (venv-lc-google) heikohotz-macbookpro:vertexai heikohotz$ pytest tests/integration_tests/test_model_garden.py tests/integration_tests/test_model_garden.py ...... [100%] ====================================================================================== warnings summary ======================================================================================= -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html |
integration test should be updated accordingly (to test the changes). |
What is linting? And I don't want to publish my endpoint IDs to the repo - but then how will it know which endpoints to use? I'm afraid I'm not familiar with integration testing tools (it was already a struggle to get pytest to run), so I'm not sure what needs to be done here :) |
can someone offer guidance on what needs to be done here? |
@marshmellow77 firstly set up poetry with linting and testing like explained over here https://python.langchain.com/docs/contributing/code#local-development-dependencies Then run from For running integration tests, the command would be |
Fixes #72