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

name 'light_rain' is not defined #2

Closed
jarun opened this issue May 23, 2021 · 5 comments
Closed

name 'light_rain' is not defined #2

jarun opened this issue May 23, 2021 · 5 comments

Comments

@jarun
Copy link

jarun commented May 23, 2021

getting the following error:

$ wwy -c kolkata  
Traceback (most recent call last):
  File "/usr/local/bin/wwy", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/wwy/__main__.py", line 92, in main
    get_ascii(info)
  File "/usr/local/lib/python3.8/dist-packages/wwy/__main__.py", line 69, in get_ascii
    for i in range(len(light_rain)):
NameError: name 'light_rain' is not defined

Please fix.

@jarun
Copy link
Author

jarun commented May 23, 2021

Also can you please make the place name the default positional parameter? So that -c is not required?

egargo added a commit that referenced this issue May 23, 2021
egargo added a commit that referenced this issue May 23, 2021
This reverts commit 9608b88.
egargo added a commit that referenced this issue May 23, 2021
egargo added a commit that referenced this issue May 23, 2021
This reverts commit 275a8a7.
egargo added a commit that referenced this issue May 23, 2021
@egargo
Copy link
Owner

egargo commented May 23, 2021

Thank you for reporting the error. I have now fixed the error. I will also be working on making the place name as the default parameter.

@egargo egargo closed this as completed May 23, 2021
@jarun
Copy link
Author

jarun commented May 24, 2021

Awesome! Neat software. All the best!

@jarun
Copy link
Author

jarun commented May 24, 2021

Would it be possible to include wind speed and the local time as well?

@egargo
Copy link
Owner

egargo commented May 24, 2021

I will try to implement that. I also wanted wwy to display the local time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants