Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include all PV in microgrid by default #457

Closed
adfarth opened this issue Dec 2, 2024 · 2 comments
Closed

Include all PV in microgrid by default #457

adfarth opened this issue Dec 2, 2024 · 2 comments
Assignees

Comments

@adfarth
Copy link
Collaborator

adfarth commented Dec 2, 2024

 if p.s.site.mg_tech_sizes_equal_grid_sizes
        @constraint(m, [t in p.techs.elec],
            m[:dvMGsize][t] == m[:dvSize][t]
        )
  • While the constraint above does get correctly applied, the value for binMGTechUsed is 0 in this case. And the result Outage.pv_microgrid_size_kw = dvMGsize * binMGTechUsed, which is why this results is 0.
  • It looks like PV is never(??) run simultaneous to a generator?! And, if the generator needs to be used at all in a given modeled outage, then it will be used to cover all of the critical load, if possible.
  • See this example in which PV is able to serve all of the load during some of the outages, and thus has a Outages.pv_microgrid_size_kw equal to the PV size: https://reopt.nrel.gov/tool/results/94d0197b-2f1a-4b3b-885c-814ecafc29ce
  • In this example, I set the max generator size to something lower than the critical load. REopt chose to add a battery (expected) and in no cases is PV serving the critical load along with the generator (unexpected): https://reopt.nrel.gov/tool/results/54f422c9-d411-41e1-99c9-81020887fd12
  • DISCOVERY: PV DOES get used in the outage when the same model is run with Xpress vs. HiGHS. See inputs for this scenario in comments
    • Potential bug, but also confusing as to why different results are obtained with Xpress: It looks like solver_is_compatible_with_indicator_constraints is false with Xpress, but is expected to be true --> made a separate issue for this: p.s.settings.solver_name not getting updated #458
@adfarth adfarth self-assigned this Dec 2, 2024
@adfarth
Copy link
Collaborator Author

adfarth commented Dec 2, 2024

mg_sizes.json

@adfarth
Copy link
Collaborator Author

adfarth commented Dec 4, 2024

Turns out this is because the fuel cost was insignificant enough to the objective function that the model can find a solution within the tolerance before finding the lower fuel cost solution that uses PV instead of the generator during outages.

@adfarth adfarth closed this as completed Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant