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

NSRDB download intermittent error #531

Closed
cpaulgilman opened this issue Feb 6, 2021 · 11 comments
Closed

NSRDB download intermittent error #531

cpaulgilman opened this issue Feb 6, 2021 · 11 comments
Assignees
Labels

Comments

@cpaulgilman
Copy link
Collaborator

Describe the bug

When NSRDB API service is down, attempt to download a weather file from Solar Resource page results in:

Could not evaluate callback function:btn_download->on_change
[507]: error: _s
[507]: failed to evaluate function call argument _d to '_s()'
[507]: failed to evaluate function call argument _d to '_s()'
[507]: eval error in statement list
[497]: eval error in statement list
[468]: eval error in statement list
[468]: error inside function call
[644]: eval error in statement list
[609]: eval error in statement list
[609]: error inside function call
[115]: eval error in statement list

To Reproduce

  1. Modify UI callback in Solar Resource Data UI form to force error from API:

image

  1. Try to download a weather file.

Expected behavior

Callback should gracefully handle API errors with a useful message.

@cpaulgilman cpaulgilman added the bug label Feb 6, 2021
@cpaulgilman cpaulgilman self-assigned this Feb 6, 2021
@cpaulgilman cpaulgilman added this to the 2020.11.29 Patch 1 milestone Feb 11, 2021
@cpaulgilman
Copy link
Collaborator Author

Also check with "advanced download."

@janinefreeman
Copy link
Collaborator

I happened to be working with advanced download today, and I'm getting the error: "The required 'email' parameter must be a valid email address". Probably a different error, but I'm assuming the fix is in the same area of the code.

@cpaulgilman
Copy link
Collaborator Author

Is this happening consistently for you in the release version of SAM 2020.11.29?

@janinefreeman
Copy link
Collaborator

Yes, seems to be. Just tried a fresh start with a different location, same error for every single file I'm attempting to download through the "advanced" window.

@cpaulgilman
Copy link
Collaborator Author

I can't seem to replicate that. What location are you trying?

@janinefreeman
Copy link
Collaborator

I tried a couple locations in CO- try "Arvada, CO"

@cpaulgilman
Copy link
Collaborator Author

Is this the error you are seeing? I got it by starting SAM and skipping the registration process. I assume you registered with email and key?

image

@janinefreeman
Copy link
Collaborator

Hmmm interesting- no, I believe I have the developer override in my registry. That's never stopped a past version from working, but seems it's on my end! I should have thought of that. Thank you!

@cpaulgilman
Copy link
Collaborator Author

Part of the issue is that the NSRDB API requires the email parameter, and we recently changed SAM to send the registration email address instead of the hard-coded SAM support email address. I'm going through the code to make sure SAM is being consistent about what email address it uses for this purpose.

Can you try registering SAM to see if the downloads work when you register with an email address?

@janinefreeman
Copy link
Collaborator

Yes, it fixes the issue for me! Thanks!

@cpaulgilman
Copy link
Collaborator Author

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

No branches or pull requests

3 participants