Skip to content

[Graph][Tensor] remove unnecessary synchronization#374

Merged
yaoyaoding merged 3 commits intohidet-org:mainfrom
xiaocenxiaocen:remove-unnecessary-synchronization
Nov 15, 2023
Merged

[Graph][Tensor] remove unnecessary synchronization#374
yaoyaoding merged 3 commits intohidet-org:mainfrom
xiaocenxiaocen:remove-unnecessary-synchronization

Conversation

@xiaocenxiaocen
Copy link
Copy Markdown
Contributor

No description provided.

@xiaocenxiaocen xiaocenxiaocen force-pushed the remove-unnecessary-synchronization branch from e25ce76 to 2da3e62 Compare November 2, 2023 19:44
@yaoyaoding
Copy link
Copy Markdown
Member

Hi @xiaocenxiaocen,

The link to default stream doc seems has broken, and the change does not pass our lint checking.

Hint: follow here to lint the code.

@xiaocenxiaocen
Copy link
Copy Markdown
Contributor Author

Hi @xiaocenxiaocen,

The link to default stream doc seems has broken, and the change does not pass our lint checking.

Hint: follow here to lint the code.

done

@yaoyaoding
Copy link
Copy Markdown
Member

Hi @xiaocenxiaocen,

I still can not open this link:

https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html\?highlight\=default%20stream\#default-stream

There is an extra \ after index.html.

@xiaocenxiaocen
Copy link
Copy Markdown
Contributor Author

Hi @xiaocenxiaocen,

I still can not open this link:

https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html\?highlight\=default%20stream\#default-stream

There is an extra \ after index.html.

Sorry for the oversight. Now fixed.

@yaoyaoding
Copy link
Copy Markdown
Member

Thanks @xiaocenxiaocen !

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