Sienna-Platform / Sienna-Platform/PowerOperationsModels.jl

Consolidate approach to models with RHS time series.

Open
#43 1 comment 0 reactions 2 assignees View on GitHub

@luke-kiernan is already working on this.

Since Feb 27, 2026.

Dominant language
Julia
Stars
2
Forks
1
Avg merge
1d 16h
Merged PRs (30d)
26

Description

In the current code base we have mixed methods on how time series are added to a model to constraint on the right hand side for ThermalStandard (derating), Interfaces, Interchanges, etc. Loads have yet another approach.

After a refactor for DLRs, the approach should be consolidated as implemented here

https://github.com/NREL-Sienna/PowerSimulations.jl/pull/1553

Roughly the approach is

  • Add a time series parameter if the attribute is added to the device model
  • when adding the Ub/lb constraints or the bounds then call a method to add the constraints with floats or with parameters.

This will avoid unnecessary duplications of constraints.

This is a large undertaking so we will split the work between @luke-kiernan and me.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.