Commit f6e3d564 authored by Turner, Sean's avatar Turner, Sean
Browse files

Fix network diagram hydropower indicator for non-powered reservoirs 



The hydropower flag in _build_topology used hasattr(attrs, "min_power_pool")
which was True for all reservoirs. Now checks max(hpf_p) > 0, correctly
marking non-powered reservoirs (zero-power HPF tables) as non-hydro.
Also adds confidentiality section to CLAUDE.md covering generated outputs.

Co-Authored-By: default avatarClaude Opus 4.6 <noreply@anthropic.com>
parent f9c19904

CLAUDE.md

+4 −0
Original line number Diff line number Diff line
@@ -8,6 +8,10 @@ Powersheds is a hydropower cascade simulator combining Rust performance with a P 


**Status**: v0.1 beta; v1.0 scheduled for September 2026.



## Confidentiality



The `examples/private/` directory contains proprietary data for private hydropower systems. This directory is excluded from version control via `.gitignore`. **Never** commit, push, or include any data, file paths, reservoir names, parameter values, or other details from `examples/private/` in the public repository — this includes CLAUDE.md, test fixtures, regression baselines, commit messages, PR descriptions, generated outputs (diagnostics HTML, plots, notebook cell outputs), and any other artifacts.



## Build Commands



```bash

python/powersheds/diagnostics.py

+1 −1
Original line number Diff line number Diff line
@@ -111,7 +111,7 @@ def _build_topology(cascade_data): 
            "name": name,

            "type": "reservoir",

            "capacity": getattr(attrs, "capacity", None),

            "hydropower": hasattr(attrs, "min_power_pool"),

            "hydropower": max(getattr(attrs, "hpf_p", [0])) > 0,

        })

    for name in confluences:

        nodes.append({"name": name, "type": "confluence"})