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

fixed configuration #13

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

fixed configuration #13

wants to merge 3 commits into from

Conversation

Rasesh2005
Copy link

Changed to work on Headless platforms as well, and fixed some configurations and typo in README.md

@pramishp
Copy link
Owner

pramishp commented Dec 19, 2024

Thank for the suggested changes and fixes in README.md . It's better to define headless in the configuration file than as a variable inside the .py file.

Also, keeping a different install.sh for 11.8 and 12 is better to support more platforms. Sth like, install-cuda11_8.sh and install-cuda12_1.sh.

@Rasesh2005
Copy link
Author

right, will do that..

@Rasesh2005
Copy link
Author

About the configuration file, should I keep it in yaml or json or simple init.py file where we can put all constants ??

@pramishp
Copy link
Owner

I prefer the yaml configuration file.

@Rasesh2005
Copy link
Author

Added the settings.yaml file

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