Paul Grosu

Results 43 comments of Paul Grosu

@abalter I think @slagelwa is referring that in the code the command parser is missing the `network` parameter. If you look at this, it does not parse for the `network`...

@moschetti You mean not possible because it's not implemented in the Life Sciences API, or that GCP would not allow it? The Life Sciences API below seems to have an...

@moschetti Of course you can have multi-region DataFlows via standard functional programming methodologies when it is data-driven with the global Cloud Logging, as one DataFlow in one region can launch...

@moschetti Not all buckets are created equal ;) [Regional buckets](https://cloud.google.com/storage/docs/storage-classes#legacy) provide [huge cost-savings over multi-region ones](https://cloud.google.com/storage/pricing#storage-pricing), which is why one would prefer that the code co-locate accordingly to those sites....

You might just need to export "CLOUDSDK_PYTHON_SITEPACKAGES=1". See: [https://cloud.google.com/sdk/crypto#cloudsdk_python_sitepackages1](https://cloud.google.com/sdk/crypto#cloudsdk_python_sitepackages1)

@mr-c But you have the [SchemaDefinedType](https://github.com/common-workflow-language/common-workflow-language/blob/e831c206023e611e462146580fc8a0f07186167f/v1.0/salad/schema_salad/metaschema/metaschema.yml#L206-L225), which gets implemented by the [SaladRecordSchema](https://github.com/common-workflow-language/common-workflow-language/blob/e831c206023e611e462146580fc8a0f07186167f/v1.0/salad/schema_salad/metaschema/metaschema.yml#L242-L277) for this. Am I not correct?

@nurmians Still looking through the code - and looked at the [release notes](https://github.com/google/deepvariant/releases) - though still reasoning through it in case it might have been implemented via some alternate logic....

Hi Pi-Chuan, I was thinking about the nice upcoming changes a bit. It will probably be easier to consolidate the two proposed flags into only one parameter such as `--non_recombinant_regions`...

Hi Andrew, I think we are in perfect agreement. Supporting what the community wants is my preferred approach as well. I was only suggesting it after having performed a reduction...

@seandavi You mean something like this via the `--print-rdf` argument through `cwltool`? ``` Bash $ cwltool --print-rdf metadata-custom.cwl @prefix CommandLineBinding: . @prefix CommandLineTool: . @prefix CommandOutputBinding: . @prefix Dirent: ....