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

Add database interface #20

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

josephmckinsey
Copy link
Contributor

@josephmckinsey josephmckinsey commented Feb 4, 2025

  • replaces Begin serialization to SQLite #13

  • Updates the existing database entries to better reflect our database design so far

  • Exports database tables to SQLite statements

  • Adds SQLite database creation logic to Julia

  • Adds initial serialization logic to Julia for objects

  • Tests database creation and insertion with ACBus.

Caveats:

  • OpenAPI objects do not have a public interface on iterating through properties, so we do that
    by converting to JSON-style Dicts. Unfortunately the only way to convert to Dict seems to be passing through strings first.
  • We avoid the use of any metaprogramming as much as possible by maintaining lists of column name changes. We should consider changing them on the JSON schema if necessary. An example is from which is a keyword which should be replaced with by from_id in all Arc related objects on the database.

@josephmckinsey josephmckinsey force-pushed the jm/add_database_interface branch from 46ff36d to 0f06905 Compare February 6, 2025 16:34
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.

1 participant