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

fix: adds pyenv to setup script #8

Merged
merged 22 commits into from
Jan 16, 2024
Merged

fix: adds pyenv to setup script #8

merged 22 commits into from
Jan 16, 2024

Conversation

JSeam2
Copy link
Contributor

@JSeam2 JSeam2 commented Jan 15, 2024

Changes:

  1. Adds pyenv to dependency script to make setup of python unix system agnostic. This has the added benefit of not corrupting the global environment python version.

@JSeam2 JSeam2 requested a review from ethan-crypto January 15, 2024 11:12
ethan-crypto

This comment was marked as outdated.

@ethan-crypto ethan-crypto self-requested a review January 15, 2024 14:34
@ethan-crypto
Copy link
Contributor

ethan-crypto commented Jan 16, 2024

Changes:

  1. Adds pyenv to dependency script to make setup of python unix system agnostic. This has the added benefit of not corrupting the global environment python version.

For now I am gonna handle all of the runner environment setup in the yaml file and not the bash script.

We also benchmark memory usage for EZKL and switch to a self-hosted-runner.

@ethan-crypto ethan-crypto merged commit 2d5e1be into main Jan 16, 2024
1 check 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