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

DB updates for Procedural Constraints #1572

Open
dandelany opened this issue Sep 26, 2024 · 0 comments · May be fixed by #1596
Open

DB updates for Procedural Constraints #1572

dandelany opened this issue Sep 26, 2024 · 0 comments · May be fixed by #1596
Assignees
Labels
constraints Anything related to the constraints domain database Anything related to the database

Comments

@dandelany
Copy link
Collaborator

dandelany commented Sep 26, 2024

Background

Now that Procedural Scheduling has been released, we are working on implementing Procedural Constraints. The workflow for using them will be very similar to scheduling goals: write a constraint in Java, compile a JAR, upload it via the Aerie UI, and add it to a plan using the same list UI that is used for eDSL constraints. Whereas procedural scheduling goals create activities on an editable plan, procedural constraints will return violations, ie. windows of time when the constraint is violated, and will not be able to edit the plan directly.

Implementing procedural goals required some changes to the database schema, to support storing them in the database and associating them with a plan:

deployment/hasura/migrations/Aerie/10_procedural_scheduling/up.sql

In particular, since multiple invocations of the same goal are now allowed in a plan spec, we had to switch from a primary key of goal_id to goal_invocation_id, representing a single invocation of a goal.

Requirements

  • Define a new database schema & write a migration similar (nearly identical) to the one for procedural scheduling, to support procedural constraints
  • Just like goals:
    • Multiple invocations of the same constraint will be allowed on a plan, necessitating a switch to constraint_invocation_id
    • Procedural constraints may have parameters/arguments
    • User should be able to have a mix of eDSL constraints & procedural constraints on their plan spec

Related Issues

#1571, NASA-AMMOS/aerie-ui#1478

@dandelany dandelany added constraints Anything related to the constraints domain database Anything related to the database labels Sep 26, 2024
@github-project-automation github-project-automation bot moved this to Todo in Aerie Sep 26, 2024
@skovati skovati moved this from Todo to In Progress in Aerie Oct 9, 2024
@skovati skovati linked a pull request Nov 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
constraints Anything related to the constraints domain database Anything related to the database
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

3 participants