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

Dynamic Simulation - Change relation between ModelVariableSetEntity and ModelVariableDefinitionEntity from Composition to Aggregation #61

Conversation

thangqp
Copy link
Contributor

@thangqp thangqp commented May 31, 2023

Change relation of ModelVariableSetEntity and ModelVariableDefinitionEntity from Composition to Aggregation for the reason of reuse variable definitions in different variable sets

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

94.4% 94.4% Coverage
0.0% 0.0% Duplication

@thangqp thangqp merged commit 3788ce0 into main Jun 1, 2023
@thangqp thangqp deleted the dynamic_simulation_change_to_many_to_many_between_variable_set_and_variable_definition branch June 1, 2023 07:23
thangqp added a commit that referenced this pull request Jul 10, 2023
…ns (#65)

- Change relation of ModelEntity and ModelParameterDefinitionEntity from Composition to Aggregation for the reason of reuse parameter definitions in different models
- Provide CRUD endpoints for injecting parameter definitions into a model
- Rework on bootstrap script sql IEEE14

**Bonus** : Some small Hibernate-related rectifications on ModelVariableDefinitionEntity and ModelVariableSetEntity entities which are following to the previous PR: #61

**Note**  : due to the lack of data migration.. the fastest way to migrate data in the current deployment is truncate all tables then rerun the script IEEE14Models.sql. The truncate tables are done in the liquibase script of this PR
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

Successfully merging this pull request may close these issues.

2 participants