New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Some small fixes to correlations and crosstab #15

Merged

cmungall merged 2 commits into master from crosstab-verbose

Feb 8, 2021

Collaborator

matentzn commented Feb 8, 2021

added verbose option to correlations (there is a lot being printed there -> would you want those tuples in the end to be somehow exported?
change order of click options (duplicate -o/--output entries).

matentzn added 2 commits

February 8, 2021 12:14


          Some small fixes to correlations and crosstab

9278bb8


          Update requirements.txt

a404d73

cmungall reviewed

View reviewed changes

sssom/cli.py

@@ @@ -114,38 +113,41 @@ def crosstab(input, output, transpose, fields): @@
               @main.command()
               @click.option('-o', '--output')
               @click.option('-t', '--transpose/--no-transpose', default=False)
+              @click.option('-v', '--verbose/--no-verbose', default=False)

Collaborator

cmungall Feb 8, 2021

there is a way in click to make verbose a global option.. we could do this.. or it may just complicate

cmungall approved these changes

View reviewed changes

cmungall merged commit 03c2ab2 into master

matentzn deleted the crosstab-verbose branch

July 9, 2021 12:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet