New database structure #3
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…s at IPP Changes to the database structure; many changes to methods and attributes name
… CRUDThermalEvent
* test: ✅ Add tests for thermal_event_instance * test: ✅ Add tests for thermal_events * test: ✅ Add tests for thermal_events and submodule crud
…the timestamp of the instance with the maximum temperature
A value of 0 is more conservative than the previous default value of 1
…t_by_experiment_id compared to get_by_columns
…ects after database commit
…simplification if polygon is short enough
The random compatibility matrix sometimes generated empty rows, and the compatibility was not ensured in test_thermal_event_get_by_experiment_id_line_of_sight
…, to push NULL values to the database by default
…dress of a user Also add associated tests (and modify existing tests to accomodate the users' email address)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request implements the changes in the database discussed between CEA and IPP, with the aim of creating a unified database of thermal events.