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 : disable system() calls #6198

Merged
merged 1 commit into from Mar 21, 2024
Merged

tests : disable system() calls #6198

merged 1 commit into from Mar 21, 2024

Conversation

ggerganov
Copy link
Owner

python and node are not always present - need to figure out a better way to run these tests

@ggerganov ggerganov merged commit 924ce1d into master Mar 21, 2024
56 of 63 checks passed
@ggerganov ggerganov deleted the gg/disable-system-calls branch March 21, 2024 14:20
@ggerganov
Copy link
Owner Author

cc @ochafik

@cebtenzzre
Copy link
Collaborator

It should be simple enough to check if std::system("python --version") and std::system("node --version") succeed before running their respective tests, and emit a warning otherwise.

ochafik added a commit to ochafik/llama.cpp that referenced this pull request Mar 21, 2024
@ochafik
Copy link
Collaborator

ochafik commented Mar 21, 2024

Sorry about the breakage and thx for the suggestion ( → drafted #6207 )

ggerganov pushed a commit that referenced this pull request Mar 22, 2024
* json: only attempt python & node schema conversion tests if their bins are present

Tests introduced in #5978
disabled in #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 1, 2024
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
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
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

3 participants