-
Notifications
You must be signed in to change notification settings - Fork 0
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
Updates for daily loading, kibo, sqlalchemy 2. #14
Conversation
…lalchemy-2 * 'sqlalchemy-2' of github.com:desihub/specprod-db: add test notebook
…lalchemy-2 * 'sqlalchemy-2' of github.com:desihub/specprod-db: update notes
PS, some additional testing is needed to ensure that any tag resulting from this PR can support If |
Also make sure to keep the tests in #12 in mind. To recap:
In other words, if we have to add columns for Finally, we may need to preserve an environment that contains SQLAlchemy 1.x in order to work with |
Note: this PR, changes the size of |
I tested |
This PR:
convert()
method to database objects (ORM classes). Basically each table knows how to convert data tables into database objects.kibo
.daily
.daily
exposure and tile files based on another specprod.Note that there is some remaining work to be done:
convert()
method. This should be included in this PR.UPDATE
sense). This may or may not end up being included in this PR.