Skip to content
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

tests: conditional python & node json schema tests #6207

Merged
merged 4 commits into from Mar 22, 2024

Conversation

ochafik
Copy link
Collaborator

@ochafik ochafik commented Mar 21, 2024

Runs these tests either when python/node is detected (as suggested by @cebtenzzre) or when LLAMA_PYTHON_AVAILABLE / LLAMA_NODE_AVAILABLE env var is set (which is done - w/ extra node & python setup - in one test env, ubuntu-focal-make, to guarantee those tests are run somewhere).

(tests introduced in #5978 / disabled in #6198 as node/python aren't always there)

@ochafik ochafik changed the title JSON schema tests: only attempt python & node tests if corresp. interpreter is found tests: conditional python & node json schema tests Mar 22, 2024
@ochafik ochafik marked this pull request as ready for review March 22, 2024 11:04
@ggerganov ggerganov merged commit f77a8ff into ggerganov:master Mar 22, 2024
46 checks passed
hodlen pushed a commit to hodlen/llama.cpp that referenced this pull request Apr 1, 2024
* json: only attempt python & node schema conversion tests if their bins are present

Tests introduced in ggerganov#5978
disabled in ggerganov#6198

* json: orange warnings when tests skipped

* json: ensure py/js schema conv tested on ubuntu-focal-make

* json: print env vars in test
hodlen pushed a commit to hodlen/llama.cpp that referenced this pull request Apr 3, 2024
* json: only attempt python & node schema conversion tests if their bins are present

Tests introduced in ggerganov#5978
disabled in ggerganov#6198

* json: orange warnings when tests skipped

* json: ensure py/js schema conv tested on ubuntu-focal-make

* json: print env vars in test
tybalex pushed a commit to tybalex/function.cpp that referenced this pull request Apr 17, 2024
* json: only attempt python & node schema conversion tests if their bins are present

Tests introduced in ggerganov#5978
disabled in ggerganov#6198

* json: orange warnings when tests skipped

* json: ensure py/js schema conv tested on ubuntu-focal-make

* json: print env vars in test
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.

None yet

2 participants