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.
Reworks the database interface on the python side using SQLModel ORM.
Should allow both for easier use of the datasets without needing to write/read/parse raw SQL commands (Both on our end when creating new datasets, and by users wanting to use the open-source datasets that are not familiar with/don't want to use SQL).
Syntax should also be easier to read what tables exist and what data they contain. As the syntax is similar to a dataclass. Ex:
vs reading the following SQL commands to try and figure out what data exists, the type of the data, and relationships. And also trying to not make mistakes when creating it in the first place
The format of the database itself should be identical. Thus making it backwards compatible with everything we already have.
All the old code also currently remains. The old database interface is located under
databaseswhile the new interface/code is underdatabases_v2.In the future if we prefer, we can deprecate the old version and replace it with this. But for now, both still exist.
Reworked the database creation scripts to use the new database_v2 interface. The created product should remain identical, thus still being backwards compatible.
Previous database creation scripts have been placed in a directory labeled "deprecated" for now. Can be removed now if desired, or at a future date when no longer needed
Future TODO: Apply the same refactoring to the Nuget .net DB as it is currently the only
create_*script that has not been reworkedAdded script to generate MinGW database.