-
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
Added csv parser to Tabledoc #275
Conversation
- Allow to add other types of entries to the triplestore that are not datasets. Ex: samples, models, instruments, people, projects... - Renamed list_data_iris() to search_iris(). It can now be use to search for all types of entries. - Renamed prepare() to as_jsonld() and made it part of the public API
For more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #275 +/- ##
==========================================
+ Coverage 78.46% 78.66% +0.20%
==========================================
Files 21 21
Lines 2206 2222 +16
==========================================
+ Hits 1731 1748 +17
+ Misses 475 474 -1 ☔ 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.
Verify that test works
tests/dataset/test_tabledoc.py
Outdated
# if True: | ||
def test_parse_csv(): | ||
"""Test parsing a csv file.""" |
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.
The output of this tests seems wrong?
@prefix dcat: <http://www.w3.org/ns/dcat#> .
@prefix emmo: <https://w3id.org/emmo#> .
<https://he-matchmaker.eu/data/sem/SEM_cement_batch2> a dcat:Dataset,
emmo:EMMO_194e367c_9783_4bf5_96d0_9ad597d48d9a .
<https://he-matchmaker.eu/data/sem/SEM_cement_batch2/77600-23-001> a dcat:Dataset,
emmo:EMMO_194e367c_9783_4bf5_96d0_9ad597d48d9a .
<https://he-matchmaker.eu/data/sem/SEM_cement_batch2/77600-23-001/77600-23-001_5kV_400x_m001> a dcat:Dataset,
emmo:EMMO_194e367c_9783_4bf5_96d0_9ad597d48d9a .
<https://he-matchmaker.eu/sample/SEM_cement_batch2/77600-23-001> a dcat:Dataset,
emmo:EMMO_194e367c_9783_4bf5_96d0_9ad597d48d9a .
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.
The output of this tests seems wrong?
Well spottet! It is corrected now.
Co-authored-by: Tor S. Haugland <torshaugland@gmail.com>
Co-authored-by: Tor S. Haugland <torshaugland@gmail.com>
Description
Added csv parser to Tabledoc (using the built-in csv module).
Note: This PR builds on top of PR #273
Type of change
Checklist for the reviewer
This checklist should be used as a help for the reviewer.