Skip to content

Quit and clean when TVM is interrupted#3640

Merged
tqchen merged 1 commit intoapache:masterfrom
ghostplant:cleanly_quit
Aug 5, 2019
Merged

Quit and clean when TVM is interrupted#3640
tqchen merged 1 commit intoapache:masterfrom
ghostplant:cleanly_quit

Conversation

@ghostplant
Copy link
Contributor

This PR is working for #3602 except when user override sys.excepthook.

@tqchen
Copy link
Member

tqchen commented Jul 29, 2019

@ghostplant can you please fix the lint error as per ci?

@tqchen tqchen added the status: need update need update based on feedbacks label Jul 29, 2019
@ghostplant ghostplant force-pushed the cleanly_quit branch 4 times, most recently from 12c1509 to 00c6243 Compare July 29, 2019 18:43
@ghostplant
Copy link
Contributor Author

@tqchen Hi, can you have a look at the lint problem (not-callable)?

@tqchen
Copy link
Member

tqchen commented Aug 1, 2019

You might want to add a pylint comment to disable the warning, as you have explicitly checked whether the attribute exists

@ghostplant
Copy link
Contributor Author

@tqchen Passed.

@tqchen tqchen merged commit cb7050c into apache:master Aug 5, 2019
@tqchen tqchen added status: accepted and removed status: need update need update based on feedbacks labels Aug 5, 2019
@tqchen
Copy link
Member

tqchen commented Aug 5, 2019

Thanks @ghostplant

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants