Potential misconfiguration in estimating `max_hours` for hydropower plants

I am wondering if the configuration here should be swapped?

This is the current code for the two settings for how to estimate max_hours for hydropower plants:

if hydro_max_hours == "energy_capacity_totals_by_country":
    # watch out some p_nom values like IE's are totally underrepresented
    max_hours_country = (
        e_missing / hydro.loc[missing_mh_i].groupby("country").p_nom.sum()
      )

elif hydro_max_hours == "estimate_by_large_installations":
    max_hours_country = (
        hydro_stats["E_store[TWh]"] * 1e3 / hydro_stats["p_nom_discharge[GW]"]
    )

hydro_stats contains data from country totals. So it would make more sense to switch and/or provide some more documentation about this?

Start in scripts/add_electricity.py around line 523 and compare the two hydro_max_hours branches with the source and meaning of hydro_stats. Verify whether the settings are reversed, then update the configuration or its documentation as appropriate; it is done when each option clearly matches the data it uses and the behavior is unambiguous.