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

Correct path notation for Windows in documentation #2171

Merged
merged 1 commit into from
Jul 4, 2023

Conversation

KaerMorh
Copy link
Contributor

@KaerMorh KaerMorh commented Jul 4, 2023

Summary

This pull request is intended to correct an issue found in the project's documentation. The previous version of the documentation used Unix-style "./" notation for file paths, which could potentially confuse or cause issues for Windows users who are expected to use ".".

Changes Made

  • Replaced all instances of "./" path notation with "." in the relevant areas of the documentation.

Justification

Many developers use Windows systems and the previous documentation would have led to incorrect path usage. With this change, the instructions in the documentation become clearer and more accurate for Windows users, preventing possible confusion and path-related issues.

Please consider merging this pull request to ensure the documentation is correct and clear for all users, irrespective of their operating system. I look forward to your feedback.

Fixed an error in the documentation where the incorrect path notation was used for Windows systems. The documentation had paths denoted with "./", which is incorrect for Windows. This has been corrected to ".\", the appropriate notation for Windows.

This change ensures that Windows users correctly follow the instructions in the documentation without encountering any path-related issues.
Copy link
Contributor

@tancik tancik left a comment

Choose a reason for hiding this comment

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

Lgtm

@tancik tancik merged commit 18ab50d into nerfstudio-project:main Jul 4, 2023
4 checks 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.

None yet

2 participants