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

Start a Migration Guide #44

Open
FranklandJack opened this issue Nov 23, 2022 · 3 comments
Open

Start a Migration Guide #44

FranklandJack opened this issue Nov 23, 2022 · 3 comments
Labels
pi DPC++ PI requirement

Comments

@FranklandJack
Copy link
Contributor

As part of Unified Runtime we need to provide a migration guide documenting the mapping from the PI plugin API to UR. This should consist of a set of mappings between the various entry points, types enumerations etc. and any divergences in semantics.

i.e. something like

Functions

PI UR Differences
piDeviceGetInfo urDeviceGetInfo None
... ... ...
... ... ...

Types

PI UR Differences
pi_context ur_context_handle_t None
... ... ...
... ... ...

Enumerations

PI UR Differences
pi_device_info ur_device_info (list of removed queries)
... ... ...
... ... ...

The initial proposal is to do this in markdown and have a living document in this repo that can be update as spec changes are made.

@FranklandJack FranklandJack self-assigned this Nov 23, 2022
@kbenzie
Copy link
Contributor

kbenzie commented Nov 23, 2022

The handwritten portions of the spec are reStructureText. I feel like this should be part of the spec document rather than separate. That implies writing the migration guide in reStructuredText not Markdown.

@FranklandJack
Copy link
Contributor Author

Sure reStructureText works just as well for me. @kbenzie Are you happy with the laying this out in tables like above? rst has better support for tables so it's probably a better choice anyway if we are going to go with that.

@kbenzie
Copy link
Contributor

kbenzie commented Nov 23, 2022

The layout seems fine.

@kbenzie kbenzie added the pi DPC++ PI requirement label Dec 5, 2022
@FranklandJack FranklandJack removed their assignment Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pi DPC++ PI requirement
Projects
None yet
Development

No branches or pull requests

2 participants