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
Rename DLContext to DLDevice #62
Conversation
Thanks @icemelon9 ! This PR is being merged |
Thanks! |
/*! \brief The device context of the tensor */ | ||
DLContext ctx; | ||
/*! \brief The device of the tensor */ | ||
DLDevice device; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tqchen @icemelon9 Unfortunately this breaks existing C code. Could you please revert the field name to ctx
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@leofang this is intentional to keep the code consistent. The existing codebase can either stay at previous version, or upgrade the code to move to the new version. Exchange won't be affected by the version being used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah OK, thanks. I failed to see this implication earlier.
This is the PR for the RFC #61 .
As discussed in the RFC, we will rename
DLContext
toDLDevice
and provideDLContext
as an alias via typedef.cc: @tqchen @junrushao1994 @leofang @szha @kkraus14