IntelPython / dpctl

Python SYCL bindings and SYCL-based Python Array API library
https://intelpython.github.io/dpctl/
Apache License 2.0
99 stars 29 forks source link

InfoDev review of API section #1659

Closed bdmoore1 closed 5 months ago

bdmoore1 commented 5 months ago

I reviewed the API section. There were some terms that did not match up so I changed them. For example C-API is not how Python uses the C API name so I removed the - .

I will submit separate PRs for these sections: beginners_guides, contributor_guides, and user_guides.

bdmoore1 commented 5 months ago

@ndgrigorian @xaleryb I do not have the ability to add reviewers. Please add appropriate reviewers to this PR.

ndgrigorian commented 5 months ago

@ndgrigorian @xaleryb I do not have the ability to add reviewers. Please add appropriate reviewers to this PR.

I've added myself and @oleksandr-pavlyk

coveralls commented 5 months ago

Coverage Status

coverage: 87.892%. remained the same when pulling 3deca077c1e0d04bbe32182a25e990479d377c90 on bdmoore1:bdmoore1-api-reference-doc-review into 6ae9f3d6f906dc9d4ff0c63a86d67687d60eea4a on IntelPython:master.

bdmoore1 commented 5 months ago

@oleksandr-pavlyk yes we should standardize the name Python/C API. I will create a DOC Jira and target 2025.0 to complete that task.