Setting prefix explicitly in excelparser

ontopy/excelparser.py

@@ -9,10 +9,11 @@
 
 Note that correct case is mandatory.
 """
-from typing import Tuple, Union, Sequence
+from typing import Tuple, Union
 import warnings
 
 import pandas as pd
+import numpy as np
 import pyparsing
 
 import ontopy
@@ -99,18 +100,19 @@ def create_ontology_from_excel(  # pylint: disable=too-many-arguments
     """
     # Get imported ontologies from optional "Imports" sheet
     if not imports:
-        imports = []
+        imports = pd.DataFrame()
+
     try:
-        imports_frame = pd.read_excel(
+        imports = pd.read_excel(
             excelpath, sheet_name=imports_sheet_name, skiprows=[1]
         )
     except ValueError:
         pass
     else:
-        # Strip leading and trailing white spaces in path
-        imports.extend(
-            imports_frame["Imported ontologies"].str.strip().to_list()
-        )
+        # Strip leading and trailing white spaces in paths
+        imports = imports.replace(" ", "")
+        # Set empty strings to nan
+        imports = imports.replace(r"^\s*$", np.nan, regex=True)
 
     # Read datafile TODO: Some magic to identify the header row
     conceptdata = pd.read_excel(
@@ -131,7 +133,7 @@ def create_ontology_from_excel(  # pylint: disable=too-many-arguments
 def create_ontology_from_pandas(  # pylint:disable=too-many-locals,too-many-branches,too-many-statements,too-many-arguments
     data: pd.DataFrame,
     metadata: pd.DataFrame,
-    imports: list,
+    imports: pd.DataFrame,
     base_iri: str = "http://emmo.info/emmo/domain/onto#",
     base_iri_from_metadata: bool = True,
     catalog: dict = None,
@@ -386,7 +388,7 @@ def get_metadata_from_dataframe(  # pylint: disable=too-many-locals,too-many-bra
     metadata: pd.DataFrame,
     base_iri: str,
     base_iri_from_metadata: bool = True,
-    imports: Sequence = (),
+    imports: pd.DataFrame = None,
     catalog: dict = None,
 ) -> Tuple[ontopy.ontology.Ontology, dict]:
     """Create ontology with metadata from pd.DataFrame"""
@@ -409,12 +411,24 @@ def get_metadata_from_dataframe(  # pylint: disable=too-many-locals,too-many-bra
     # Add imported ontologies
     catalog = {} if catalog is None else catalog
     locations = set()
-    for location in imports:
+    for _, row in imports.iterrows():
+        # for location in imports:
+        location = row["Imported ontologies"]
         if not pd.isna(location) and location not in locations:
             imported = onto.world.get_ontology(location).load()
             onto.imported_ontologies.append(imported)
             catalog[imported.base_iri.rstrip("#/")] = location
             locations.add(location)
+        # set defined prefix
+        if not pd.isna(row["prefix"]):
+            # set prefix for all ontologies with same 'base_iri_root'
+            if not pd.isna(row["base_iri_root"]):
+                onto.set_common_prefix(
+                    iri_base=row["base_iri_root"], prefix=row["prefix"]
+                )
+            # If base_root not given, set prefix only to top ontology
+            else:
+                imported.prefix = row["prefix"]
 
     with onto:
         # Add title

tests/testonto/excelparser/catalog-v001.xml

@@ -2,5 +2,6 @@
 <catalog prefer="public" xmlns="urn:oasis:names:tc:entity:xmlns:xml:catalog">
     <group id="Folder Repository, directory=, recursive=true, Auto-Update=false, version=2" prefer="public" xml:base="">
         <uri name="https://raw.githubusercontent.com/emmo-repo/emmo-repo.github.io/master/versions/1.0.0-beta/emmo-inferred-chemistry" uri="https://raw.githubusercontent.com/emmo-repo/emmo-repo.github.io/master/versions/1.0.0-beta/emmo-inferred-chemistry2.ttl"/>
+        <uri name="http://ontology.info/ontology" uri="imported_onto/ontology.ttl"/>
     </group>
 </catalog>

tests/testonto/excelparser/fromexcelonto.ttl

@@ -12,7 +12,8 @@
         "Jesper Friis"@en,
         "Sylvain Gouttebroze"@en ;
     term:title "A test domain ontology"@en ;
-    owl:imports <https://raw.githubusercontent.com/emmo-repo/emmo-repo.github.io/master/versions/1.0.0-beta/emmo-inferred-chemistry> ;
+    owl:imports <http://ontology.info/ontology>,
+        <https://raw.githubusercontent.com/emmo-repo/emmo-repo.github.io/master/versions/1.0.0-beta/emmo-inferred-chemistry> ;
     owl:versionInfo "0.01"@en .
 
 :EMMO_0264be35-e8ad-5b35-a1a3-84b37bde22d1 a owl:Class ;
@@ -25,6 +26,10 @@
         :EMMO_4b32833e-0833-56a7-903c-28a6a8191fe8 ;
     core:prefLabel "FiniteTemporalPattern"@en .
 
+:EMMO_080262b7-4f7e-582b-916e-8274c73dd629 a owl:Class ;
+    rdfs:subClassOf <http://ontology.info/ontology#testclass> ;
+    core:prefLabel "ANewTestClass"@en .
+
 :EMMO_1c81f1eb-8b94-5e74-96de-1aeacbdb5b93 a owl:Class ;
     emmo:EMMO_967080e5_2f42_4eb2_a3a9_c58143e835f9 "The boundary of a grain"@en ;
     rdfs:subClassOf :EMMO_472ed27e-ce08-53cb-8453-56ab363275c4 ;
@@ -58,6 +63,10 @@
         :EMMO_9fa9ca88-2891-538a-a8dd-ccb8a08b9890 ;
     core:prefLabel "FiniteSpatioTemporalPattern"@en .
 
+:EMMO_e4e653eb-72cd-5dd6-a428-f506d9679774 a owl:Class ;
+    rdfs:subClassOf <http://ontology.info/subontology#testclass2> ;
+    core:prefLabel "AnotherNewTestClass"@en .
+
 :EMMO_e633d033-2af6-5f04-a706-dab826854fb1 a owl:Class ;
     emmo:EMMO_967080e5_2f42_4eb2_a3a9_c58143e835f9 "The boundary of a subgrain"@en ;
     rdfs:subClassOf owl:Thing ;

tests/testonto/excelparser/imported_onto/catalog-v001.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<catalog prefer="public" xmlns="urn:oasis:names:tc:entity:xmlns:xml:catalog">
+    <group id="Folder Repository, directory=, recursive=true, Auto-Update=false, version=2" prefer="public" xml:base="">
+	    <uri name="http://ontology.info/ontology/0.1.0" uri="./ontology.ttl"/>
+	    <uri name="http://ontology.info/ontology/0.1.0/subontology" uri="./subontology.ttl"/>
+    </group>
+</catalog>

tests/testonto/excelparser/imported_onto/ontology.ttl

@@ -0,0 +1,18 @@
+@prefix : <http://ontology.info/ontology#> .
+@prefix owl: <http://www.w3.org/2002/07/owl#> .
+@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
+@prefix xml: <http://www.w3.org/XML/1998/namespace> .
+@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .
+@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
+@prefix skos: <http://www.w3.org/2004/02/skos/core#> .
+@base <http://ontology.info/ontology> .
+
+<http://ontology.info/ontology> rdf:type owl:Ontology ;
+                             owl:versionIRI <http://ontology.info/ontology/0.1.0> ;
+                             owl:imports <http://ontology.info/ontology/0.1.0/subontology> ;
+			     owl:versionInfo "0.1.0" .
+
+
+:testclass rdf:type owl:Class ;
+    rdfs:subClassOf owl:Thing ;
+    skos:prefLabel "TestClass"@en .

tests/testonto/excelparser/imported_onto/subontology.ttl

@@ -0,0 +1,21 @@
+@prefix : <http://ontology.info/subontology#> .
+@prefix owl: <http://www.w3.org/2002/07/owl#> .
+@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
+@prefix xml: <http://www.w3.org/XML/1998/namespace> .
+@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .
+@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
+@prefix skos: <http://www.w3.org/2004/02/skos/core#> .
+@base <http://ontology.info/subontology> .
+
+<http://ontology.info/subontology> rdf:type owl:Ontology ;
+    owl:versionIRI <http://ontology.info/ontology/0.1.0/subontology> .
+
+
+# Annotations
+skos:prefLabel rdf:type owl:AnnotationProperty .
+skos:altLabel rdf:type owl:AnnotationProperty .
+
+
+:testclass2 rdf:type owl:Class ;
+    rdfs:subClassOf owl:Thing ;
+    skos:prefLabel "TestClass2"@en .