Skip to content

[WIP] Check cupy.available#2967

Closed
niboshi wants to merge 1 commit intochainer:masterfrom
niboshi:check-cupy-available
Closed

[WIP] Check cupy.available#2967
niboshi wants to merge 1 commit intochainer:masterfrom
niboshi:check-cupy-available

Conversation

@niboshi
Copy link
Copy Markdown
Member

@niboshi niboshi commented Jul 4, 2017

(See: cupy/cupy#247)

Check cupy.available when setting chainer.cuda.available, so that it will be False when there is no available device (by CUDA_VISIBLE_DEVICES=-1)

cupy/cupy#247 should be merged first.

@niboshi niboshi added the cat:enhancement Implementation that does not break interfaces. label Jul 4, 2017
@niboshi niboshi force-pushed the check-cupy-available branch from 65a6724 to 40b58b3 Compare July 11, 2017 16:50
@niboshi niboshi force-pushed the check-cupy-available branch from 40b58b3 to d69a027 Compare July 11, 2017 17:26
@niboshi niboshi changed the title Check cupy.available [WIP] Check cupy.available Jul 13, 2017
@okuta okuta self-assigned this Jul 13, 2017
@niboshi
Copy link
Copy Markdown
Member Author

niboshi commented Jul 24, 2017

I have a trouble handling memory pool initialization (https://github.com/niboshi/chainer/blob/d69a027ddcf19c71cc386ebd0e3c88aa76d398a8/chainer/cuda.py#L132), which is currently done as module-level code. If we delay availability determination, I have no idea when they should be done.

@stale
Copy link
Copy Markdown

stale bot commented Oct 23, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Not updated for a longer period of time. label Oct 23, 2017
@stale stale bot removed the stale Not updated for a longer period of time. label Oct 24, 2017
@niboshi niboshi added the st:needs-discussion State indicating that discussions are needed before proceeding. label Oct 24, 2017
@stale
Copy link
Copy Markdown

stale bot commented Jan 21, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Not updated for a longer period of time. label Jan 21, 2018
@stale
Copy link
Copy Markdown

stale bot commented Feb 20, 2018

This issue is closed as announced. Feel free to re-open it if needed.

@stale stale bot closed this Feb 20, 2018
@niboshi niboshi added this to the Closed issues and PRs milestone Feb 20, 2018
@niboshi niboshi deleted the check-cupy-available branch February 20, 2018 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cat:enhancement Implementation that does not break interfaces. st:needs-discussion State indicating that discussions are needed before proceeding. stale Not updated for a longer period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants