Fix #99: "RuntimeError: This event loop is already running" in colab and notebook. Re-PR of #100

[aaraney]

This is a resubmission of #100 with minor changes to the unit test added. Fixes #99.

Changes

• If event loop is already running and RestClient is instantiated, try to import nest_asyncio and patch with nest_asyncio.apply(). Raise ModuleNotFoundError if nest_asyncio not installed.

Testing

1. Test added to verify ModuleNotFoundError is raised. If nest_asyncio is installed, test fails.

Checklist

• [x] PR has an informative and human-readable title
• [x] PR is well outlined and documented. See #12 for an example
• [x] Changes are limited to a single goal (no scope creep)
• [x] Code can be automatically merged (no conflicts)
• [x] Code follows project standards (see CONTRIBUTING.md)
• [x] Passes all existing automated tests
• [x] Any change in functionality is tested
• [x] New functions are documented (with a description, list of inputs, and expected output) using numpy docstring formatting
• [x] Placeholder code is flagged / future todos are captured in comments
• [x] Reviewers requested with the Reviewers tool :arrow_right:

[jarq6c]

@aaraney Have you confirmed this change fixes the original google collab issue? Our other software devs may have an opinion on the general pattern, but nothing appears particularly obscene about this contribution to me.

[aaraney]

@aaraney Have you confirmed this change fixes the original google collab issue? Our other software devs may have an opinion on the general pattern, but nothing appears particularly obscene about this contribution to me.

Yes. Here is a colab notebook that verifies this PR resolves the previous issue.

[jarq6c]

Thanks for the fix!

[aaraney]

For sure!