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

Tutorial on how to install Myria3D on WSL2 #53

Merged
merged 5 commits into from
Jan 23, 2023
Merged

Conversation

jistiak
Copy link
Contributor

@jistiak jistiak commented Jan 17, 2023

For Windows users it is quite tricky to get everything working in Myria3D. It's best to use Windows Subsystem for Linux (WSL2) to get the most out of this library if you want to work on Windows. While setting up WSL and installing Myria3D is not that difficult, if you want to utilize GPU, it becomes much more challenging. This guide tries to address the necessary steps that needs to be taken to get a properly working environment with GPU support.

For Windows users it is quite tricky to get everything working in Myria3D. It's best to use Windows Subsystem for Linux (WSL2) to get the most out of this library if you want to work on Windows. While setting up WSL and installing Myria3D is not that difficult, if you want to utilize GPU, it becomes much more challenging. This guide tries to address the necessary steps that needs to be taken to get a properly working environment with GPU support.
@CharlesGaydon
Copy link
Collaborator

Hi @jistiak, and thank you very much to offer to contribute to the package!
This seems like a nice addition to facilitate onboarding from Windows users. I'll be able to review your proposal at the end of the week :)

Copy link
Collaborator

@CharlesGaydon CharlesGaydon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice contribution for Windows user, thanks! I made a few suggestions to add some external references.

Additionally, your page needs to be referenced in the documentation tree (index page) to be accessible :)

Copy link
Collaborator

@CharlesGaydon CharlesGaydon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small comment left and we are good to merge :)

Copy link
Collaborator

@CharlesGaydon CharlesGaydon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ! Thank you again for this first external contribution :)

@CharlesGaydon CharlesGaydon merged commit 113da17 into IGNF:main Jan 23, 2023
@CharlesGaydon
Copy link
Collaborator

Note: there seem to be an issue in documentation build, due to an invalid environment creation (pyg issue pyg-team/pytorch_geometric#2040). I'll look into it.

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