-
Notifications
You must be signed in to change notification settings - Fork 30
516 dynamic tables and columns #718
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
Open
Simon-Will
wants to merge
37
commits into
OpenEnergyPlatform:develop
Choose a base branch
from
Simon-Will:516-dynamic-tables-and-columns
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
4a0effd
Add draft for dynamic table creation
Simon-Will 7cafcf4
Continue with implementing XSD reading
Simon-Will 10f957c
Continue with implementation
Simon-Will b954457
Continue
Simon-Will 4704ca6
Rename mastr_2.py to mastr.py
Simon-Will 783f2b2
Get to working state
Simon-Will 0aa8335
Make a couple small adjustments
Simon-Will fc5bfec
Implement date-based docs download
Simon-Will 49b07e0
Make wrong XML schema a bit easier to see
Simon-Will 1f6a670
Implement adding a custom primary key column
Simon-Will 774b7ad
Add basic CSV export
Simon-Will f31622c
Implement translation feature
Simon-Will f743960
Generate SQLAlchemy core tables, not ORM models
Simon-Will 4e248bf
Add functions for formatting SQLAlchemy tables
Simon-Will 26899b8
Fix bug where docs download failed for non-given URL (due to typo)
Simon-Will 661d980
Start fixing tests
Simon-Will 9e1aecc
Add docstrings
Simon-Will 5c7e5be
Remove unused code
Simon-Will 2158b8a
Make code improvements (unused imports, etc.)
Simon-Will bd76ef1
Make existing tests work
Simon-Will 2dac26b
Fix bug with primary keys
Simon-Will 09cc1e0
Add tests for english & data model generation
Simon-Will cb932df
Update documentation
Simon-Will 4fc054b
Add Simon Will to authors
Simon-Will bfb5aa4
Add missing conftest.py file
Simon-Will ba69e75
Improve clarity around artificial primary keys
Simon-Will e214d98
Add changelog entry
Simon-Will 9b92e57
Add views for old table names + a few other things
Simon-Will 6bb869e
Merge remote-tracking branch 'upstream/develop' into 516-dynamic-tabl…
Simon-Will c40fb9a
Remove some unused code
Simon-Will 21e7ce7
Fix test
Simon-Will f3e2333
Address review comments
Simon-Will 8c6a1db
516: Add test for XSD fallback
Simon-Will 3d2f1da
Let open-MaStR logger propagate and handle with root logger
Simon-Will 91e8ee2
Merge remote-tracking branch 'upstream/develop' into 516-dynamic-tabl…
Simon-Will 60b9e77
516: Run black
Simon-Will 3082862
516: Merge branch 'develop' into 516-dynamic-tables-and-columns
Simon-Will File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We discussed in a call and @nesnoj pointed out if we can use a LLM to generate an extended changelog - since a lot of stuff happend here. I did this and it created this new changelog:
[v0.xx.x] Unreleased - 202x-xx-xx
Added
Mastr.generate_data_modelmethod that downloads the current MaStR documentation and generates SQLAlchemy tables from the XSD definitions; supportsenglish=Truefor English columnnames
#718
mastr_table_to_db_tableargument toMastr.downloadto pass a custom database schema#718
alter_database_tablesargument toMastr.downloadto prevent open-mastr from issuing DDL statements#718
Changed
#718
#718
#718
Removed
Mastr.translate; English table and column names are now available via theenglish=Trueparameter ingenerate_data_modelanddownload#718