mwvgroup / pittgoogle-user-demos

GNU General Public License v3.0
2 stars 0 forks source link

Switch `pgb-broker-utils` -> `pittgoogle-client` #20

Closed troyraen closed 2 months ago

troyraen commented 1 year ago

Closes #5

Depends on mwvgroup/pittgoogle-client#19

Changes:

Capturing some to-dos:

troyraen commented 11 months ago

I'd like to consider putting all resource names in a yaml file, then updating setup.sh and main.py to read them from the yaml.

Benefits:

Other thoughts and implementation details to consider:

If we go forward with this it should probably be a separate PR.

hernandezc1 commented 11 months ago

This PR is extremely insightful as to how I should write code moving forward. The changes in main.py make the file readable--this is something I really appreciate.

My next task will be to review and keep an eye on PR #19 to understand how things are working under the hood.

troyraen commented 2 months ago

This is working. Going to merge and continue iterating.