broadinstitute/fiss

config output to terminal not easily readable

Closed this issue · 5 comments

For example, note that JSON here is returned in 1, long wrapping line. It would be much more readable & usable to see the line broken up, similar to the way pretty_print might.

%  fissfc config_get -c mRNA_Preprocess

{"name":"mRNA_Preprocess","methodRepoMethod":{"methodName":"preprocess_mrna","methodVersion":12,"methodNamespace":"broadgdac","methodUri":"agora://broadgdac/preprocess_mrna/12","sourceRepo":"agora"},"outputs":{"preprocess_mrna_workflow.preprocess_mrna.mRNA_antisense":"this.mRNA_antisense","preprocess_mrna_workflow.preprocess_mrna.mRNA_protein_coding":"this.mRNA_protein_coding","preprocess_mrna_workflow.preprocess_mrna.mRNA_lincRNA":"this.mRNA_lincRNA","preprocess_mrna_workflow.preprocess_mrna.mRNA_protein_coding_zscore":"","preprocess_mrna_workflow.preprocess_mrna.mRNA_others":"this.mRNA_others","preprocess_mrna_workflow.preprocess_mrna.mRNA_mirna":"this.mRNA_mirna","preprocess_mrna_workflow.preprocess_mrna.mRNA_pseudogene":"this.mRNA_pseudogene","preprocess_mrna_workflow.preprocess_mrna.mRNA_all":"this.mRNA_all","preprocess_mrna_workflow.preprocess_mrna.preprocess_mrna_pkg":"this.pkg__mRNA_Preprocess"},"inputs":{"preprocess_mrna_workflow.preprocess_mrna.mrnafile":"this.mRNA__geneExpNormed__FPKM","preprocess_mrna_workflow.preprocess_mrna.package":"workspace.package","preprocess_mrna_workflow.preprocess_mrna.mapfile":"\"gs://broad-institute-gdac/reference/genome/genCode-v22.Genecode2GenetypeMap.txt\"","preprocess_mrna_workflow.preprocess_mrna.ram_gb":"","preprocess_mrna_workflow.preprocess_mrna.outputprefix":"this.name"},"rootEntityType":"sample_set","prerequisites":{},"methodConfigVersion":1,"deleted":false,"namespace":"broadgdac"}

Fixed in 04423b3, will close this once other features are added to the branch and we're ready to do a PR.

Yeah, it looks like what I was referring to in my OP never made it beyond a dev version I must've squirreled away and apparently deleted, so thanks for doing this. What other features, may I ask?

@sammeier has several that he wants to put in.

@sammeier, can you elaborate on David's point?

David, how about you PR this branch? It's been sitting 10 days and I could've used the more readable configs the last 2 days (with all the attribute/config_get etc debugging that was done)