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

Modify simulation config model to include executed field and update id -> simulation_id #39

Open
jearistiz opened this issue May 2, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request refactor Source code needs to be modified to meet new criteria

Comments

@jearistiz
Copy link
Member

jearistiz commented May 2, 2021

Modify the simulation config model to include executed field

{
  id -> simulation_id : ObjectId
  (··· all other fields)
  executed: bool
}
@jearistiz jearistiz self-assigned this May 2, 2021
@jearistiz jearistiz added the enhancement New feature or request label May 2, 2021
@jearistiz jearistiz changed the title Modify simulation config model to include executed field Modify simulation config model to include executed field and update id field -> simulation_id May 2, 2021
@jearistiz jearistiz added the refactor Source code needs to be modified to meet new criteria label May 3, 2021
@jearistiz jearistiz changed the title Modify simulation config model to include executed field and update id field -> simulation_id Modify simulation config model to include executed field and update id -> simulation_id May 3, 2021
@Camilo-HG
Copy link
Member

Camilo-HG commented May 3, 2021

After reviewing the diagram flow, we have noticed that instead of executed it sould be Status and it can take the next values: Incomplete config, Running, Done and Error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor Source code needs to be modified to meet new criteria
Projects
None yet
Development

No branches or pull requests

2 participants