Skip to content

Update branch to latest version python 3.11 support only#32

Merged
EdmondIguazio merged 4 commits into1.11.xfrom
main
Dec 30, 2025
Merged

Update branch to latest version python 3.11 support only#32
EdmondIguazio merged 4 commits into1.11.xfrom
main

Conversation

@Matankl
Copy link
Collaborator

@Matankl Matankl commented Dec 30, 2025

Update branch to latest version python 3.11 support only

Matankl and others added 4 commits December 23, 2025 17:16
* from datasets import Dataset, load_dataset
* import mlrun
* import pandas as pd

- remove if sys.version_info.major == 3 and sys.version_info.minor == 9:
    requirements += ['protobuf==3.20.3'] to stop the support of python 3.9
* from datasets import Dataset, load_dataset
* import mlrun
* import pandas as pd
* from mlrun.features import Feature
* import mlrun.common.schemas.alert as alert_constants

- In llm-monitoring-main.ipynb
* remove if sys.version_info.major == 3 and sys.version_info.minor == 9:
    requirements += ['protobuf==3.20.3'] to stop the support of python 3.9
* change the dependency's in the image to be lighter
*

- In requirements.txt change deepeval==3.7.0

- In project_setup.py change requirements of generate_ds and remove

- In generate_ds.py remove "if hf_repo_id:" and dataset import, it causes dependency problem

- in llm-monitoring-main.ipynb remove hf_repo_id parameter to "generate_ds" func, "train" func and "llm-server" func

- In llm_as_a_judge.py remove the huggingface part, there is no need for it and it makes the requirements to big and causes dependency problems

- In train.py instead of getting and filtering the dataset in the func and causes dependency issues it now receiving it as a parameter
- Remove py3.9 support in project_setup.py  "if sys.version_info.major == 3 and sys.version_info.minor == 9:"
fixing the demo and adjusting it to python 3.11 support only
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@EdmondIguazio EdmondIguazio merged commit 835cc92 into 1.11.x Dec 30, 2025
2 checks passed
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.

2 participants