Sai Srinivas Tangudu
Sai Srinivas Tangudu
Thanks, I'll make required changes and let you know.
@gpsaggese I’ve addressed all the TODOs—let me know if there’s anything I might’ve missed or needs further changes.
> I believe this was already merged by merging master. Pls confirm and we can close Yes GP.
@gpsaggese I have few doubts in hopenai.py 1. Why we are doing ```isinstance()``` for ```type_=="is_string"``` in ```convert_to_type()```? ``` def convert_to_type(col, type_): if type_ == "is_bool": return col.map( lambda x: isinstance(x,...
@gpsaggese Please review the PR [#777](https://github.com/causify-ai/helpers/pull/777).
@gpsaggese Please review the PRs [HelpersTask727_Generalise_hopenai.py_to_use_openrouter_interface](https://github.com/causify-ai/helpers/pull/736) [HelpersTask727_Querying_openrouter_models](https://github.com/causify-ai/helpers/pull/737) [HelpersTask727_Generalise_price_counting_interface_in_hopenai.py](https://github.com/causify-ai/helpers/pull/738)
> [ ] Compatible with the replay mechanism (there should be no change besides maybe storing the name of the model or some other metadata) A separate PR is not...
@gpsaggese Seems like there is no "openai" library in docker environment. ``` mypy-extensions 1.0.0 nbclient 0.10.2 nbconvert 7.16.6 nbformat 5.10.4 nest-asyncio 1.6.0 networkx 3.4.2 notebook 7.3.2 notebook_shim 0.2.4 numpy 2.2.3...
I ran my code in the Docker container successfully, and similarly, I attempted to run the test cases using the "pytest" command in the Docker bash. However, it’s throwing the...
1. Correct 2. Correct 3. Yes, i mean outside the container after adding in virtual env. 4. This is the entire log. ```(venv) user_501@5e625c348160:/app$ pytest helpers/test/test_hopenai.py ImportError while loading conftest...