Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[Hardware][Neuron] Refactor neuron support #3471
[Hardware][Neuron] Refactor neuron support #3471
Changes from all commits
f2c1047
774dc8f
faf3007
936b213
11f3e0b
ae4bafe
16a3603
0bd8f83
dcf5a9a
d3fd29b
637de02
8aa8c6c
04f9c77
364d620
3972fc4
7673c1d
de2d54c
ac3e9b1
f1cc16c
bd2c1f4
14e09b6
dcf0f59
3a414e7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Instead of string comparison, set a
property
in device_config?FYI, we might consider some device-specific config with
device="neuron config=(a=1, b=2, c=3)"
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.
device_type
is different fromdevice
. I think we can safely assumedevice_type
can only becuda
andneuron
for now? We can put the config string in other fields ofdevice_config
.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.
Sure, i think we are fine to move on with this.