Skip to content

Commit e6f1c18

Browse files
Eyal-Danielidanielperezziguazio-cicdguylei-codeamitnGiniApps
authored
Cherry Pick from Development (#957)
* replace author to Iguazio manually (#905) * Organize CLI directory + new CLI for generating item.yaml files (#906) * create a CLI for generating item.yaml and organize the CLI directory * modify comments to module * PR fixes * Update cli/common/generate_item_yaml.py Co-authored-by: Eyal Danieli <eyal_danieli@mckinsey.com> --------- Co-authored-by: Eyal Danieli <eyal_danieli@mckinsey.com> * fill count events notebook (#908) * avoid noise reduction unit test (#909) * Add histogram-data-drift monitoring application module (without example) (#911) * histogram data drift module with empty example notebook * post review fixes * chore(readme): auto-update asset tables [skip ci] * Fill histogram-data-drift example notebook (#912) * fill data-drift nb * post review fixes * Add evidently demo app monitoring application module (without example) (#913) * sphinx build docs bug fix * add evidently demo app module (empty example notebook) * post review changes * chore(readme): auto-update asset tables [skip ci] * [Translate] Require torch>=2.6 for the translate function to work properly (#915) * lock torch valid version * edit the item.yaml and generated function.yaml * update mlrun version * [CLI] Generated READMEs are produced with broken links to the items (#918) * fix * test fix * test fix * test fix * test fix * final workflow * chore(readme): auto-update asset tables [skip ci] * OpenAI Module without notebook (#917) * First commit OpenAI Module * First commit OpenAI Module * Update example filename in item.yaml * Delete modules/src/openai_proxy/requirements.txt No need due to no unitest * Update item.yaml for OpenAI application configuration * Update modules/src/openai_proxy/openai.py Co-authored-by: Daniel Perez <100069700+danielperezz@users.noreply.github.com> * Change category name from 'GenAI' to 'genai' * Update package requirements with version constraints * Second commit adding notebook * Refactor OpenAI proxy to use base64 encoded script Refactor OpenAI proxy implementation to use base64 encoded script and update FastAPI app configuration. * Change deployment method to OpenAIModule * Third commit adding notebook * Third commit adding notebook * Remove package requirements from item.yaml Removed specific requirements for fastapi and requests. * Rename item and update kind in YAML * Update openai.py * Third commit adding notebook * Fix after review * Fix after review --------- Co-authored-by: Daniel Perez <100069700+danielperezz@users.noreply.github.com> * chore(readme): auto-update asset tables [skip ci] * [Evidently] Fill example notebook (#919) * add notebook + rename directory + correct evidently version * remove extra cell * chore(readme): auto-update asset tables [skip ci] * chore(readme): auto-update asset tables [skip ci] * [CLI + Modules] Fix time format in generate item yaml script (#922) * fix time format for evidently and hist * fix cli script * fix datetime format * chore(readme): auto-update asset tables [skip ci] * chore(readme): auto-update asset tables [skip ci] * Fix CMD first commit * Fix CMD second commit * remove max-width restriction from the main content (#929) * add test, requirement file and notebook * fix cli/utils/helpers.py * Add schema verification step (#955) * step files, empty notebook * missing type hint * fix test * post review fixes * fix test and cover new argument * add copyright * [UV] Convert project to UV (#949) * convert project to uv * update mlrun ver * improve readme * improve readme * remove troubleshooting from readme * improve README.md * improve README.md * replcae lint with ruff and update according to review * adjust make test to work on all assets * update to use python 3.11 (required for tests) * fix according to review * fix readmes * add steps readme * add steps readme * add steps readme * add missing notebook (#956) * delete openai example * delete openai example --------- Co-authored-by: Daniel Perez <100069700+danielperezz@users.noreply.github.com> Co-authored-by: iguazio-cicd <iguaziocicd@gmail.com> Co-authored-by: guylei-code <guyleibu@gmail.com> Co-authored-by: amitnGiniApps <amitn@gini-apps.com>
1 parent e6af51a commit e6f1c18

File tree

14 files changed

+1020
-325
lines changed

14 files changed

+1020
-325
lines changed

.gitignore

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,3 +141,8 @@ dmypy.json
141141
conda-setup-cpu
142142
conda-setup-gpu
143143
.DS_Store
144+
145+
# UV
146+
.uv/
147+
uv.lock
148+

Makefile

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
.PHONY: help sync format lint test cli
2+
3+
help: ## Show this help message
4+
@echo 'Usage: make [target]'
5+
@echo ''
6+
@echo 'Available targets:'
7+
@awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z_-]+:.*?## / {printf " %-15s %s\n", $$1, $$2}' $(MAKEFILE_LIST)
8+
9+
sync: ## Sync dependencies from lockfile
10+
uv sync
11+
12+
format: ## Format code with ruff
13+
uv run ruff format .
14+
uv run ruff check --fix .
15+
16+
lint: ## Run linters
17+
uv run ruff format --check .
18+
uv run ruff check .
19+
20+
test: ## Run tests for a specific asset (usage: make test NAME=aggregate [TYPE=functions])
21+
@if [ -z "$(NAME)" ]; then \
22+
echo "Error: NAME parameter is required"; \
23+
echo "Usage: make test NAME=<asset_name> [TYPE=functions|modules|steps]"; \
24+
echo "Example: make test NAME=aggregate"; \
25+
echo "Example: make test NAME=mymodule TYPE=modules"; \
26+
exit 1; \
27+
fi
28+
@TYPE=$${TYPE:-functions}; \
29+
echo "Running tests for $$TYPE/src/$(NAME)"; \
30+
uv run python -m cli.cli run-tests -r $$TYPE/src/$(NAME) -s py -fn $(NAME)
31+
32+
cli: ## Run the CLI tool (usage: make cli ARGS="command args")
33+
uv run python -m cli.cli $(ARGS)
34+
35+
.DEFAULT_GOAL := help
36+

0 commit comments

Comments
 (0)