Skip to content
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.

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

vignette_from_args fails #49

Open
dimalvovs opened this issue Oct 6, 2023 · 0 comments
Open

vignette_from_args fails #49

dimalvovs opened this issue Oct 6, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@dimalvovs
Copy link
Contributor

to reproduce:

(.env) u88@clarity-3 pycogaps % python vignette_from_args.py 

______      _____       _____   ___  ______  _____ 
| ___ \    /  __ \     |  __ \ / _ \ | ___ \/  ___|
| |_/ /   _| /  \/ ___ | |  \// /_\ \| |_/ /\ `--. 
|  __/ | | | |    / _ \| | __ |  _  ||  __/  `--. |
| |  | |_| | \__/\ (_) | |_\ \| | | || |    /\__/ /
\_|   \__, |\____/\___/ \____/\_| |_/\_|    \____/ 
       __/ |                                       
      |___/             
                                 
                    
pycogaps version  0.0.1
This vignette was built using pycogaps version 0.0.1
Traceback (most recent call last):
  File "/Users/u88/Documents/jhu/cogaps/pycogaps/vignette_from_args.py", line 16, in <module>
    params_file = sys.argv[1]
IndexError: list index out of range

and also if the params.yaml file is provided as input:

(.env) u88@clarity-3 pycogaps % python vignette_from_args.py params.yaml 

______      _____       _____   ___  ______  _____ 
| ___ \    /  __ \     |  __ \ / _ \ | ___ \/  ___|
| |_/ /   _| /  \/ ___ | |  \// /_\ \| |_/ /\ `--. 
|  __/ | | | |    / _ \| | __ |  _  ||  __/  `--. |
| |  | |_| | \__/\ (_) | |_\ \| | | || |    /\__/ /
\_|   \__, |\____/\___/ \____/\_| |_/\_|    \____/ 
       __/ |                                       
      |___/             
                                 
                    
pycogaps version  0.0.1
This vignette was built using pycogaps version 0.0.1
Traceback (most recent call last):
  File "/Users/u88/Documents/jhu/cogaps/pycogaps/vignette_from_args.py", line 20, in <module>
    os.mkdir(outdir)
OSError: [Errno 30] Read-only file system: '/output/'

Lets:

  1. fail more gracefully (invite user to specify params.yaml) if its missing;
  2. fix the reason for failure even if params file is provided.
@dimalvovs dimalvovs added the bug Something isn't working label Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant