Skip to content

Commit

Permalink
Merge branch 'develop' into ghp-district
Browse files Browse the repository at this point in the history
  • Loading branch information
atpham88 committed Sep 21, 2024
2 parents bf6711e + 5084565 commit a7ca525
Show file tree
Hide file tree
Showing 5 changed files with 548 additions and 537 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ jobs:
matrix:
julia-version: ['1.8']
julia-arch: [x64]
# os: [ubuntu-latest, windows-latest, macOS-11]
os: [windows-latest, macOS-11]
os: [windows-latest]

steps:
- uses: actions/checkout@v2
Expand All @@ -24,4 +23,4 @@ jobs:
- uses: julia-actions/julia-buildpkg@latest
# - uses: mxschmitt/action-tmate@v3 # for interactive debugging
- run: julia --project=. -e 'using Pkg; Pkg.activate("test"); Pkg.rm("Xpress"); Pkg.activate("."); using TestEnv; TestEnv.activate(); cd("test"); include("runtests.jl")'
shell: bash
shell: bash
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,13 @@ Classify the change according to the following categories:
### Deprecated
### Removed

## Develop 08-09-2024
### Changed
- Improve the full test suite reporting with a verbose summary table, and update the structure to reflect long-term open-source solver usage
- Removed MacOS from the runner list and just run with Windows OS, since MacOS commonly freezes and gets cancelled. We have not seen Windows OS pass while other OS's fail. .
- Suppress JuMP warning messages from 15-minute and multiple PVs test scenarios to avoid flooding the test logs with those warnings
- Updated/specified User-Agent header of "REopt.jl" for PVWatts and Wind Toolkit API requests; default before was "HTTP.jl"; this allows specific tracking of REopt.jl usage which call PVWatts and Wind Toolkit through api.data.gov.

## v0.47.2
### Fixed
- Increased the big-M bound on maximum net metering benefit to prevent artificially low export benefits.
Expand Down
2 changes: 1 addition & 1 deletion src/core/production_factor.jl
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ function get_production_factor(wind::Wind, latitude::Real, longitude::Real, time
resource = []
try
@info "Querying Wind Toolkit for resource data ..."
r = HTTP.get(url; retries=5)
r = HTTP.get(url, ["User-Agent" => "REopt.jl"]; retries=5)
if r.status != 200
throw(@error("Bad response from Wind Toolkit: $(response["errors"])"))
end
Expand Down
2 changes: 1 addition & 1 deletion src/core/utils.jl
Original file line number Diff line number Diff line change
Expand Up @@ -461,7 +461,7 @@ function call_pvwatts_api(latitude::Real, longitude::Real; tilt=latitude, azimut

try
@info "Querying PVWatts for production factor and ambient air temperature... "
r = HTTP.get(url, keepalive=true, readtimeout=10)
r = HTTP.get(url, ["User-Agent" => "REopt.jl"]; keepalive=true, readtimeout=10)
response = JSON.parse(String(r.body))
if r.status != 200
throw(@error("Bad response from PVWatts: $(response["errors"])"))
Expand Down
Loading

0 comments on commit a7ca525

Please sign in to comment.