broadinstitute / fiss

FireCloud Service Selector (FISS) -- Python bindings and CLI for FireCloud execution engine
BSD 3-Clause "New" or "Revised" License
29 stars 25 forks source link

adding fields parameter to get_workspace() #146

Closed luke-c-sargent closed 4 years ago

luke-c-sargent commented 4 years ago

Hello,

much like PR #143, this adds the fields parameter to get_workspace(), allowing firecloud to return selected-for data instead of a JSON dump of all info. i have tested the API call, and have added fields to calls in fiss.py where i thought they could be useful; some of the commands are unfamiliar to me, so I am not entirely sure my implementation was perfect. examples:

in the case of both of these, i also get the same error with the fissfc command from rc-0.16.28, so my guess is the error is unrelated to the changes, but i wanted to inform any reviewers who might see something amiss.

thanks for reading, please let me know how i can improve the quality of the submission.