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

Update documentation #140

Merged
merged 3 commits into from Dec 15, 2022
Merged

Update documentation #140

merged 3 commits into from Dec 15, 2022

Conversation

crimist
Copy link
Contributor

@crimist crimist commented Dec 15, 2022

  • Add ampere verification for direct backend to docs
  • Minor restructuring to improve clarity
  • Add links to improve clarity

@crimist
Copy link
Contributor Author

crimist commented Dec 15, 2022

Does the direct backend still need the experimental warning etc? It does seem like very few people are having issues with it so I could clean it up and just add an line that says "here's an issue template if you have a problem". You're more familiar with the issue tracker though so lmk.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@elFarto
Copy link
Owner

elFarto commented Dec 15, 2022

Does the direct backend still need the experimental warning etc?

I think so. While it does appear to be working for everyone, it could easily break on the next driver upgrade. It's also not the NVIDIA supported way to do this, and relies on me guessing how the NVIDIA driver API works.

@crimist
Copy link
Contributor Author

crimist commented Dec 15, 2022

Ah I see, so it uses undocumented functions?

@elFarto
Copy link
Owner

elFarto commented Dec 15, 2022

Ah I see, so it uses undocumented functions?

That's correct.

@elFarto
Copy link
Owner

elFarto commented Dec 15, 2022

Thanks for the changes!

@elFarto elFarto merged commit acd4696 into elFarto:master Dec 15, 2022
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