David Cavazos

This might require updating to the latest branch, secrets have changed and it might be using an older image version.

After looking at the system logs for the Dataflow workers in one of the earlier runs, it looks like the workers don't have enough disk space to load the container...

`Startup of the worker pool in zone us-central1-a failed to bring up any of the desired 1 workers. Please refer to https://cloud.google.com/dataflow/docs/guides/common-errors#worker-pool-failure for help troubleshooting. ZONE_RESOURCE_POOL_EXHAUSTED_WITH_DETAILS: Instance 'dataflow-gemma-flex-templ-07171029-mk9a-harness-7qn5' creation failed:...

We need the tests to pass in order to merge.

I made some code simplifications. I think the issue with the byte64 was that Pub/Sub expects bytes and it was being passed a string. The local runner failed, but I...

``` File "/workspace/healthcare/api-client/v1/fhir/fhir_stores_test.py", line 244, in test_patch_fhir_store fhir_stores.patch_fhir_store(project_id, location, dataset_id, fhir_store_id) TypeError: patch_fhir_store() missing 1 required positional argument: 'pubsub_topic' ``` Seems like the test needs to be updated.

> This is a solid starting place. Plenty of feedback to make this more robust, but nothing that I think should stop from moving forward. I am approving, but before...

Moved tests into the source directory, updated README as well.

For the GPU examples I'll have to manually experiment with which combination of nvidia image and tensorflow version works. There are no exact matches from the docs and I've tried...

I reproduced this locally. I think `six` is not an indirect dependency anymore. I think we can simply add it to the dependencies.