-
Notifications
You must be signed in to change notification settings - Fork 2
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 SQLite compatibility with new module #190
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #190 +/- ##
==========================================
- Coverage 80.11% 79.24% -0.88%
==========================================
Files 21 29 +8
Lines 2872 3272 +400
==========================================
+ Hits 2301 2593 +292
- Misses 571 679 +108 ☔ View full report in Codecov by Sentry. |
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.
I think that this is my laste request to merge this
src/OpenSQL/README.md
Outdated
|
||
- In case of a vector attribute, a Table should be created with its name indicating the name of the Collection and the name of the attribute, separated by `_vector_`, as presented below | ||
|
||
<p style="text-align: center;"> COLLECTION_NAME_vector_ATTRIBUTE_NAME</p> |
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.
COLLECTION_vector_ATTRIBUTE
This PR allows one to create, load and modify a study using SQLite.
A new module
OpenSQL
was created and should be able to function independent from PSRI. Moreover, this new functionality should have the same interface as with the current PSRI data format.That being said, in the file
sql_interface.jl
, PSRI functions are overloaded to fit the new module's methods to PSRI's syntax.