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

[DOC] transfer dalapi #1074

Closed

Conversation

outoftardis
Copy link
Contributor

Description

  • Set up dalapi for daal repo
  • Test on array.hpp + temporary array.rst file

Do not merge.

@rlnx
Copy link
Contributor

rlnx commented Oct 14, 2020

There are some files in __pycache__ folder, add them to .gitignore

@rlnx
Copy link
Contributor

rlnx commented Oct 14, 2020

docs/include should be removed and dalapi configured to parse headers in cpp/oneapi/dal.

@outoftardis
Copy link
Contributor Author

docs/include should be removed and dalapi configured to parse headers in cpp/oneapi/dal.

I'm aware. This is just for test purposes.

@outoftardis outoftardis force-pushed the doc-transfer-spec-comments branch from 1b395b1 to 98d3b2e Compare October 22, 2020 09:05
@@ -24,6 +24,10 @@
namespace oneapi::dal {
namespace v1 {

/// @tparam T The type of the memory block elements within the array.
/// $T$ can represent any type.
Copy link
Contributor

Choose a reason for hiding this comment

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

$...$ is used for mathematical expressions, to refer to the template parameter or normal parameters of the function, use :literal:`...` . It should be changed in the spec as well

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