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

Power Gradient Constraints #1218

Open
bqth29 opened this issue Dec 6, 2024 · 2 comments · May be fixed by #1229
Open

Power Gradient Constraints #1218

bqth29 opened this issue Dec 6, 2024 · 2 comments · May be fixed by #1229
Labels
feature New feature or request good first issue Good for newcomers intertemporality Encompasses the use of multi-time-steps optimization and intertemporal constraints

Comments

@bqth29
Copy link
Collaborator

bqth29 commented Dec 6, 2024

Describe the current behavior

No response

Describe the expected behavior

As introduced in #1208 (in this comment), the intertemporal RAO will require Power Gradient Constraints (PGCs).

Implement the PowerGradientConstraint class in intertemporal-rao.

Describe the motivation

No response

Extra Information

After some discussions, we concluded that there this no need, at first, to have a start and end validity dates for the constraints (though they may be required for other types of intertemporal constraints). Thus, the PowerGradientConstraint shall only have 3 attributes: the network element, the gradient value and the variation direction.

public record PowerGradientConstraint(String networkElementId, double powerGradient, RangeAction.VariationDirection variationDirection) {
}

Besides, it could be worth to isolate the VariationDirection enum out of the RangeAction i,nterface in a standalone file in crac-api.

Wait for #1186 to be merged before to address this issue.

@bqth29 bqth29 added feature New feature or request good first issue Good for newcomers intertemporality Encompasses the use of multi-time-steps optimization and intertemporal constraints labels Dec 6, 2024
@bqth29 bqth29 added this to the Intertemporal RAO milestone Dec 6, 2024
@bqth29 bqth29 moved this to Ready in OpenRao Planning Dec 6, 2024
@Godelaine
Copy link
Collaborator

Is the reason why you want to separate VariationDirection to reuse it for other constraints ?

@bqth29
Copy link
Collaborator Author

bqth29 commented Dec 6, 2024

Is the reason why you want to separate VariationDirection to reuse it for other constraints ?

No, it is because currently, the enum is part of the RangeAction interface. As it can also be used for gradient constraints I thought it would be cleaner to get it out as it will no longer be a specific RangeAction's field.

@bqth29 bqth29 linked a pull request Dec 9, 2024 that will close this issue
7 tasks
@bqth29 bqth29 moved this from Ready to In review in OpenRao Planning Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request good first issue Good for newcomers intertemporality Encompasses the use of multi-time-steps optimization and intertemporal constraints
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants