manohar/Remodel
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: land cost calculation - filter by both id and attribution
Some checks are pending
CI / Engine — lint / typecheck / test (push) Waiting to run
Details
CI / API — lint / typecheck / test (push) Waiting to run
Details
CI / Web — typecheck / lint / build (push) Waiting to run
Details

Browse source 
- Solar land cost: only items with id containing 'land' AND attribution 'SolarOnly'
- Wind land cost: only items with id containing 'land' AND attribution 'WindOnly'
- Prevents wind land cost from appearing in pure solar projects
This commit is contained in:
Manohar Gupta 2026-05-29 18:24:02 +05:30
parent e920fa6f71
commit 17dc873d9b
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/web/components/InputsTab.tsx
View file

@ -791,7 +791,9 @@ export function InputsTab({ scenarioId, inputsJson, onSaved }: Props) {
const solarBOS = solarItems.filter(it => ["solar_inverter", "solar_dc_bos", "solar_ac_bos", "solar_hr"].includes(it.id)).reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);
const solarModule = solarItems.filter(it => it.id === "solar_module").reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);
const windWTG = windItems.filter(it => ["wind_wtg", "wind_tower", "wind_bop", "wind_e_and_c"].includes(it.id)).reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);
// Solar land: only include items with id containing "land" AND attribution "SolarOnly"
const solarLandCost = solarItems.filter(it => it.id.includes("land") && it.attribution === "SolarOnly").reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);
// Wind land: only include items with id containing "land" AND attribution "WindOnly"
const windLandCost = windItems.filter(it => it.id.includes("land") && it.attribution === "WindOnly").reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);
const bessAll = bessItems.reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);
const solarEPC = solarItems.filter(it => it.category === "EPCOverhead").reduce((acc, it) => acc + (computeItemCr(it, solarDcMwp, windMw, bessMwh, 0, effectiveLandAcres) ?? 0), 0);