Sept26 Updates Merge - #734
Open
Bill-Becker wants to merge 29 commits into
Open
Bill-Becker wants to merge 29 commits into
Bill-Becker wants to merge 29 commits into
Conversation
The julia_src environment pinned PROJ_jll = "900.0.0" as a direct
dependency. On Julia 1.13 that build fails to load:
InitError: could not load library ".../lib/libproj.so"
libcurl.so.4: version `CURL_4' not found (required by libproj.so)
PROJ_jll 900.100.0 requires versioned CURL_4 symbols that the libcurl
bundled with Julia 1.13 does not export. Because PROJ_jll was pinned
here directly, this environment kept resolving the broken build even
after REopt.jl relaxed its own ArchGDAL bound.
Move the bound to PROJ_jll = "902", which resolves:
ArchGDAL 0.9.4 -> 0.10.12
GDAL_jll 300.500.101+0 -> 304.1200.300+0
PROJ_jll 900.100.0+0 -> 902.800.100+0
SQLite_jll 3.48.0+0 -> 3.53.4+0
Also drop the SQLite_jll = "~3.48" pin. That pin existed only because
PROJ_jll 900.x records `DT_NEEDED libsqlite3.so.0` and SQLite_jll >=
3.51 changed its SONAME. SQLite_jll 3.53.4 still installs a
libsqlite3.so.0 symlink alongside libsqlite3.so.3.53.4, and the 902.x
libproj loads against it cleanly, so the constraint is no longer needed.
Verified in the julia:1.13.0 image from a clean depot: `using REopt`
and `using GhpGhx` succeed, find_ashrae_zone_city(39.7, -105.2) returns
"Boulder", avert_region_abbreviation(...) returns ("RM", 0.0), and a
full `docker build` of julia_src/Dockerfile completes through
precompile.jl.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Update the default ElectricStorage O&M cost fraction from 0.025 to 0.04 per 2025 ATB defaults update and add a migration that enforces validation bounds for om_cost_fraction_of_installed_cost and size_class.
Rename supplementary_firing_capital_cost_per_kw to supplementary_firing_installed_cost_per_mmbtu_per_hour and supplementary_firing_max_steam_ratio to supplementary_firing_max_ratio in CHPInputs. Rename size_supplemental_firing_kw to size_supplemental_firing_mmbtu_per_hour and add size_supplementary_firing_ratio and annual_supplementary_firing_thermal_production_mmbtu to CHPOutputs. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…u_per_hour, just update migration don't add one
bug fix
update tests
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge of these PRs:
and this branch for CHP supplementary firing updates: