-
Notifications
You must be signed in to change notification settings - Fork 9
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
E-Module Mapping Script #117
Comments
@raviapatel Please add the orange box for the ID, so I can append an ID after the underscore for linking mix and emodule. |
Test |
I want to review code - please let me do it =) |
@ThiloMuth if you accept the invitation to the repo, you can review the merge requests, e.g. this one |
@mattheokru Is this way of spelling ("Modul") within the e-module ontology on purpose/ predefined by the ontology or something like that? It looks german to me. |
No nothing predefined by the Ontology, I will change it. I updated the Ontologies in the pull request "updating Ontologies" |
|
I have a question regarding the emodul_metadata_extraction.py: It should create an entry for the "processedFile" key in the dictionary, having as value the path to csv file with values extracted by emodul_generate_processed_data.py. Currently as a placeholder it's a null pointer. |
Currently, we store the files locally (as you have mentioned with that path), so I would add for now exactly this path. Ultimately, this would have to go to a file server/mongdb/openBIS with a URI, but please talk to @AidaZt or @ThiloMuth on how we should then reference these files in our KG. |
Andre suggested that we can reference the link/URL to the file. |
But that links is not existing, or how do we intend to store data such that this link is actually a real reference? |
Then, we need, at least to talk about a file server, point straight to github raw files, dereferencing URIs providing RDF content, etc. I suggest we have a short meeting to have an agreement about the best way for us to deal with the raw files. Best regards, |
About the Transducer Column: Edit: @raviapatel Is this how you imagined it to be? |
The data type expected is an xsd:integer, therefore it's supposed to be an integer number. If you still not sure about the data type, then store as an xsd:string. |
And a list of integers doesn't fit the integer type, right? |
I thinks so, because we either have string or integer as a type and we can't refer it as xsd:list or something? I think for now leave it as xsd:string. |
if you still need to store a kind of list of values in RDF, there is an example here: |
For your information:
@joergfunger @raviapatel, maybe @ThiloMuth wants to have a look at it, too. |
Ok this is also fine for me |
How will the info about the openBis raw data location get mapped? Will this be defined in the mapping script or created during the metadata extraction so that the mapping script will automatically map it? @joergfunger |
That should be done during the extraction of the metadata. In the final setup, we will have the data all stored in the openBIS system (metadata), and then extracting this information together with the link to the raw data file should happen. Afterwards, the mapping script will just take that information in the metadata.json and replace that value in the ttl file obtained from the diagrams.net ttl template. |
The e-module part got separated from the rest of the CPTO. We have a working mapping script for the simple ontology and metadata to be mapped. So on this branch we will write a mapping script for the e-module. Things to do:
The text was updated successfully, but these errors were encountered: