spacetelescope / jwst_gtvt

Other
10 stars 11 forks source link

Create Verbose argument #39

Closed mfixstsci closed 4 years ago

mfixstsci commented 4 years ago

Adding choice for users to get verbose output from the app or the command line interface. Related to #38 . Default option is True to not change the default behavior of the tool but now the option in get_table exists as well as the command line argument --no_verbose to silence output.

mfixstsci commented 4 years ago

In the exoctk app, for the sanity of the exomast folks, we need to allow them to set the verbose argument to False to not clog up the logs. @jaymedina do you think you can implement those changes on the exoctk end?

jaymedina commented 4 years ago

@mfixstsci yep, I can do that. thanks!

Edit: I just remembered, we are using an old version of JWST GTVT for ExoCTK because there was an issue running the most recent version. I can't remember what the issue was because it was back in December, but I think it had to do with astroquery? Was this issue fixed?

mfixstsci commented 4 years ago

@jaymedina I believe so? Do you have a link to the github issue by chance?