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.

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

Address SAM #1968 #1998

Merged
merged 22 commits into from
Feb 2, 2025
Merged

Address SAM #1968 #1998

merged 22 commits into from
Feb 2, 2025

Conversation

sjanzou
Copy link
Collaborator

@sjanzou sjanzou commented Jan 29, 2025 •
edited
Loading

Addresses Linux issue with NSRDB and IURDB downloads in 2024.12.12 release.

Also, addresses utilities with zero rate crash on Linux.

Tested on RHEL 8.6, MacOS 15.2 and Windows 11.
image

image

sjanzou added 22 commits December 10, 2024 00:38
@sjanzou
2024.12.12.ssc.297 release candidate - no expiration
5a29814
@sjanzou
Geocode debugging on Linux
73ef931
@sjanzou
Update Geocoding URL for Linux #1968
c18ac2e
@sjanzou
Add http headers for Linux #1968
7622535
@sjanzou
timezone geocoding update for Linux #1968
13dac1e
@sjanzou
Additional bing tz debugging messages for Linux #1968
3ce73bc
@sjanzou
Merge branch 'develop' into SAM_1968
7d67af1
@sjanzou
Merge branch 'patch' into SAM_1968
e0d97a1
@sjanzou
move http headers for #1968
36ae74a
@sjanzou
api and geocode return message for #1968
9d4809c
@sjanzou
additional Linux debugging for #1968
0f304cc
@sjanzou
Try to fix RapidJSON empty document on Linux issue #1968
86365f8
@sjanzou
Test using parse stream to encode for linux #1968
ba5c60a
@sjanzou
Test escaping quotes for linux #1968
c96d4ca
@sjanzou
Testing urdb curl updates for #1968
064bcc7
@sjanzou
update parsing flags for linux #1968
5bfdf14
@sjanzou
std string parsing with curl library #1968
b253604
@sjanzou
Testing after verifying working with json str in vscode on RHEL #1968
dd1b2bf
@sjanzou
Switch from std::string to wxString for testing GeocodeDeveloper method
db66759 
#1968
@sjanzou
Rewrite all parsing for UTF16 and test on Windows #1968
128023d
@sjanzou
Merge remote-tracking branch 'origin/patch' into SAM_1968
8d4f328
@sjanzou
Remove debugging messages and handle zero rate utilities #1968
b9f16fb 
Working without issue on Windows
@sjanzou sjanzou self-assigned this Jan 29, 2025
@sjanzou sjanzou linked an issue Jan 29, 2025 that may be closed by this pull request
NREL Developer API calls do not work in Linux #1968
Closed
cpaulgilman
cpaulgilman approved these changes Jan 31, 2025
View reviewed changes
Copy link
Collaborator

@cpaulgilman cpaulgilman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for sorting this out. URDB and weather file downloads work on Linux Mint 21.1 Cinnamon.

@sjanzou sjanzou merged commit 994cff4 into patch Feb 2, 2025
8 checks passed
@sjanzou sjanzou deleted the SAM_1968 branch February 2, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cpaulgilman cpaulgilman cpaulgilman approved these changes

@janinefreeman janinefreeman Awaiting requested review from janinefreeman

Assignees

@sjanzou sjanzou

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

NREL Developer API calls do not work in Linux
2 participants
@sjanzou @cpaulgilman