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

Add initial content to the Sphinx Doc #59

Merged
merged 16 commits into from
Jan 13, 2022

Conversation

SimeonEhrig
Copy link
Member

No description provided.

- maybe not the final logo -> waiting for feedback
- removed unnecessary code
- unified examples
- improved comments
- make it single dimension only, because the API does not support
  multi dimension
@SimeonEhrig SimeonEhrig mentioned this pull request Jan 5, 2022
@SimeonEhrig SimeonEhrig force-pushed the initialDocumentation branch 2 times, most recently from 21da4d0 to e5f5606 Compare January 6, 2022 15:57
@SimeonEhrig SimeonEhrig force-pushed the initialDocumentation branch from e5f5606 to 7a5ae21 Compare January 6, 2022 16:03
- integrate it in Sphinx Doc and make it available on
  readthedocs.io
@SimeonEhrig SimeonEhrig force-pushed the initialDocumentation branch from 7f502c6 to 09d2cef Compare January 6, 2022 16:15
@SimeonEhrig SimeonEhrig marked this pull request as ready for review January 10, 2022 16:44
@SimeonEhrig SimeonEhrig mentioned this pull request Jan 10, 2022
Copy link
Member

@bernhardmgruber bernhardmgruber left a comment

Choose a reason for hiding this comment

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

I read the README.md and skimmed the rest.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@SimeonEhrig SimeonEhrig force-pushed the initialDocumentation branch 2 times, most recently from 3126907 to 6e62a6f Compare January 11, 2022 09:05
Co-authored-by: Bernhard Manfred Gruber <[email protected]>
Copy link
Member

@j-stephan j-stephan left a comment

Choose a reason for hiding this comment

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

Thanks, that looks like a great improvement. A couple remarks:

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
example/transform/src/transform-main.cpp Outdated Show resolved Hide resolved
example/transform/src/transform-main.cpp Outdated Show resolved Hide resolved
example/transform/src/transform-main.cpp Outdated Show resolved Hide resolved
example/transform/src/transform-main.cpp Outdated Show resolved Hide resolved
example/transform/src/transform-main.cpp Show resolved Hide resolved
README.md Show resolved Hide resolved
@j-stephan j-stephan merged commit 48413f4 into alpaka-group:master Jan 13, 2022
@SimeonEhrig SimeonEhrig mentioned this pull request Jan 13, 2022
@SimeonEhrig SimeonEhrig added this to the release 0.2 milestone Jan 19, 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.

4 participants