Octue

<!--- SKIP AUTOGENERATED NOTES --->
Contents

Fixes
- Ensure `MissingServiceID` error is raised when the `SERVICE_ID` environment variable is empty or missing on deployed services

<!--- END AUTOGENERATED NOTES --->

<!--- START AUTOGENERATED NOTES --->
Contents

Testing
- Improve mocking of subscription creation test

<!--- END AUTOGENERATED NOTES --->

Summary
Use concurrency/multithreading to significantly speed up downloads of a dataset's files from the cloud. Download times were measured to be almost 7 times faster on a test dataset of 30 datafiles, although this will vary depending on the number of files downloaded.

<!--- SKIP AUTOGENERATED NOTES --->
Contents

Enhancements
- Make `Dataset.download_all_files` multithreaded

Testing
- Mock Pub/Sub in subscription creation test

Other
- Improve documentation of `AppFrom` class

<!--- END AUTOGENERATED NOTES --->

Summary
Handle a `KeyError` when trying to delete `app` from `sys.modules`. This does not mean that the problem is actually fixed, simply that it is downgraded from exception to warning. This enables us to have clear logs on services whilst investigating.

<!--- START AUTOGENERATED NOTES --->
Contents

Fix
- Catch `KeyError` and issue warning for further diagnosis

<!--- END AUTOGENERATED NOTES --->

Summary
Use the name given to `Child` instances to add context to log messages and exceptions forwarded from them to the parent. Logs forwarded from two children a user has locally named `child-1` and `child-2` will appear like:

[child-1] A log message
[child-2] Another log message
[child-1] Some more


after any other context added by logging handlers.

<!--- SKIP AUTOGENERATED NOTES --->
Contents

Enhancements
- Use children's names to contextualise log messages from them
- Add a `service_name` argument to `Service.wait_for_answer` that will be added to any log messages and exceptions forwarded by the responding service. This defaults `REMOTE` if none is provided.

<!--- END AUTOGENERATED NOTES --->

Summary
Explicitly define `Analysis` attributes so that code introspection tools can more easily detect their existence. This is useful for developers and general transparency/readability of the class interface.

<!--- START AUTOGENERATED NOTES --->
Contents

Refactoring
- Define `Analysis` attributes explicitly in constructor rather than implicitly using a `for` loop and `setattr`

<!--- END AUTOGENERATED NOTES --->