lint and format

lint and format codebase with ruff and black
RDFLib · Jul 22, 2024 · e5d9b18 · e5d9b18
1 parent 715ec4b
commit e5d9b18
Show file tree

Hide file tree

Showing 35 changed files with 180 additions and 167 deletions.
diff --git a/poetry.lock b/poetry.lock
diff --git a/prez/app.py b/prez/app.py
@@ -20,7 +20,8 @@
 from prez.models.model_exceptions import (
     ClassNotFoundException,
     URINotFoundException,
-    NoProfilesException, InvalidSPARQLQueryException,
+    NoProfilesException,
+    InvalidSPARQLQueryException,
 )
 from prez.routers.catprez import router as catprez_router
 from prez.routers.cql import router as cql_router
@@ -168,7 +169,7 @@ def assemble_app(
             ClassNotFoundException: catch_class_not_found_exception,
             URINotFoundException: catch_uri_not_found_exception,
             NoProfilesException: catch_no_profiles_exception,
-            InvalidSPARQLQueryException: catch_invalid_sparql_query
+            InvalidSPARQLQueryException: catch_invalid_sparql_query,
         },
         **kwargs
     )

diff --git a/prez/models/__init__.py b/prez/models/__init__.py
@@ -1 +1 @@
-from prez.models.search_method import SearchMethod
+from prez.models.search_method import SearchMethod as SearchMethod
diff --git a/prez/models/model_exceptions.py b/prez/models/model_exceptions.py
@@ -44,4 +44,4 @@ class InvalidSPARQLQueryException(Exception):
 
     def __init__(self, error: str):
         self.message = f"Invalid SPARQL query: {error}"
-        super().__init__(self.message)
+        super().__init__(self.message)
diff --git a/prez/models/object_item.py b/prez/models/object_item.py
@@ -1,15 +1,8 @@
-from typing import Optional, FrozenSet, Tuple
-from typing import Set
+from typing import Optional, FrozenSet
 
-from pydantic import BaseModel, root_validator
+from pydantic import BaseModel
 from rdflib import URIRef, PROF
 
-from prez.cache import endpoints_graph_cache
-from prez.models.model_exceptions import ClassNotFoundException
-from prez.reference_data.prez_ns import PREZ, ONT
-from prez.services.curie_functions import get_uri_for_curie_id
-from prez.services.model_methods import get_classes
-
 
 class ObjectItem(BaseModel):
     uri: Optional[URIRef] = None

diff --git a/prez/models/profiles_item.py b/prez/models/profiles_item.py
@@ -7,7 +7,6 @@
 from prez.cache import profiles_graph_cache
 from prez.config import settings
 from prez.services.curie_functions import get_uri_for_curie_id, get_curie_id_for_uri
-from prez.services.model_methods import get_classes
 
 PREZ = Namespace("https://prez.dev/")
 

diff --git a/prez/queries/vocprez.py b/prez/queries/vocprez.py
@@ -8,37 +8,37 @@ def get_concept_scheme_query(iri: str, bnode_depth: int) -> str:
         """
         PREFIX prez: <https://prez.dev/>
         PREFIX skos: <http://www.w3.org/2004/02/skos/core#>
-        
-        CONSTRUCT {  
+
+        CONSTRUCT {
             ?iri ?p ?o .
-            
+
             {% if bnode_depth > 0 +%}
             ?iri ?p0 ?o0 .
             {% endif %}
-            
+
             {% for i in range(bnode_depth) %}
             ?o{{ i }} ?p{{ i + 1 }} ?o{{ i + 1 }} .
             {% endfor %}
-            
+
             ?iri prez:childrenCount ?childrenCount .
         }
         WHERE {
             BIND(<{{ iri }}> as ?iri)
             ?iri ?p ?o .
             FILTER (?p != skos:hasTopConcept)
-            
+
             {
                 SELECT (COUNT(?topConcept) AS ?childrenCount)
                 WHERE {
                     BIND(<{{ iri }}> as ?iri)
                     ?iri skos:hasTopConcept ?topConcept .
                 }
             }
-            
+
             {% if bnode_depth > 0 %}
             ?iri ?p0 ?o0 .
             {% endif %}
-            
+
             {% for i in range(bnode_depth) %}
             ?o{{ i }} ?p{{ i + 1 }} ?o{{ i + 1 }} .
             FILTER (isBlank(?o0))
@@ -58,7 +58,7 @@ def get_concept_scheme_top_concepts_query(iri: str, page: int, per_page: int) ->
         PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
         PREFIX skos: <http://www.w3.org/2004/02/skos/core#>
         PREFIX xsd: <http://www.w3.org/2001/XMLSchema#>
-        
+
         CONSTRUCT {
             ?concept skos:prefLabel ?label .
             ?concept prez:childrenCount ?narrowerChildrenCount .
@@ -79,27 +79,27 @@ def get_concept_scheme_top_concepts_query(iri: str, page: int, per_page: int) ->
             }
             ?iri rdf:type ?type .
             ?concept rdf:type ?conceptType .
-        
+
             {
                 SELECT (COUNT(?childConcept) AS ?childrenCount)
                 WHERE {
                     BIND(<{{ iri }}> as ?iri)
                     ?iri skos:hasTopConcept ?childConcept .
                 }
             }
-        
+
             {
                 # Using two OPTIONAL clauses with a UNION causes ?narrowConcept to be duplicated.
                 # Use DISTINCT to get an accurate count.
                 SELECT ?concept ?label (COUNT(DISTINCT ?narrowerConcept) AS ?narrowerChildrenCount)
                 WHERE {
                     BIND(<{{ iri }}> as ?iri)
-                    
+
                     {
                         OPTIONAL {
                             ?iri skos:hasTopConcept ?concept .
                             ?concept skos:prefLabel ?label .
-                
+
                             OPTIONAL {
                                 ?narrowerConcept skos:broader ?concept .
                             }
@@ -112,7 +112,7 @@ def get_concept_scheme_top_concepts_query(iri: str, page: int, per_page: int) ->
                         OPTIONAL {
                             ?concept skos:topConceptOf ?iri .
                             ?concept skos:prefLabel ?label .
-                
+
                             OPTIONAL {
                                 ?narrowerConcept skos:broader ?concept .
                             }
@@ -141,7 +141,7 @@ def get_concept_narrowers_query(iri: str, page: int, per_page: int) -> str:
         PREFIX rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
         PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
         PREFIX skos: <http://www.w3.org/2004/02/skos/core#>
-        
+
         CONSTRUCT {
             ?concept skos:prefLabel ?label .
             ?concept prez:childrenCount ?narrowerChildrenCount .
@@ -161,25 +161,25 @@ def get_concept_narrowers_query(iri: str, page: int, per_page: int) -> str:
                 ?concept skos:prefLabel ?label .
             }
             ?iri rdf:type ?type .
-            
+
             {
                 SELECT (COUNT(?childConcept) AS ?childrenCount)
                 WHERE {
                     BIND(<{{ iri }}> as ?iri)
                     ?childConcept skos:broader ?iri .
                 }
             }
-        
+
             {
                 SELECT ?concept ?label (skos:Concept AS ?conceptType) (COUNT(?narrowerConcept) AS ?narrowerChildrenCount)
                 WHERE {
                     BIND(<{{ iri }}> as ?iri)
-                    
+
                     {
                         OPTIONAL {
                             ?concept skos:broader ?iri .
                             ?concept skos:prefLabel ?label .
-                
+
                             OPTIONAL {
                                 ?narrowerConcept skos:broader ?concept .
                             }
@@ -192,7 +192,7 @@ def get_concept_narrowers_query(iri: str, page: int, per_page: int) -> str:
                         OPTIONAL {
                             ?iri skos:narrower ?concept .
                             ?concept skos:prefLabel ?label .
-                
+
                             OPTIONAL {
                                 ?narrowerConcept skos:broader ?concept .
                             }
@@ -201,7 +201,7 @@ def get_concept_narrowers_query(iri: str, page: int, per_page: int) -> str:
                             }
                         }
                     }
-                    
+
                     # Filter out any unbound ?concept rows which are invalid and may contain
                     # a count of 0. This is possible because both paths within the select
                     # query are using the OPTIONAL clause.

diff --git a/prez/renderers/json_renderer.py b/prez/renderers/json_renderer.py
@@ -1,6 +1,4 @@
-from itertools import chain
-
-from rdflib import Graph, URIRef, RDF, SH, Literal
+from rdflib import Graph, URIRef, RDF, SH
 from rdflib.term import Node
 
 from prez.cache import profiles_graph_cache

diff --git a/prez/renderers/renderer.py b/prez/renderers/renderer.py
@@ -70,7 +70,7 @@ async def return_from_graph(
     else:
         if "anot+" in mediatype:
             non_anot_mediatype = mediatype.replace("anot+", "")
-            profile_headers['Content-Type'] = non_anot_mediatype
+            profile_headers["Content-Type"] = non_anot_mediatype
             graph = await return_annotated_rdf(graph, profile, repo)
             content = io.BytesIO(
                 graph.serialize(format=non_anot_mediatype, encoding="utf-8")

diff --git a/prez/routers/management.py b/prez/routers/management.py
@@ -28,7 +28,7 @@ async def index():
     g.add((URIRef(settings.system_uri), PREZ.version, Literal(settings.prez_version)))
     g += endpoints_graph_cache
     g += await return_annotation_predicates()
-    log.info(f"Populated API info")
+    log.info("Populated API info")
     return await return_rdf(g, "text/turtle", profile_headers={})
 
 

diff --git a/prez/routers/object.py b/prez/routers/object.py
@@ -1,35 +1,11 @@
-from string import Template
-from typing import FrozenSet, Optional
-
 from fastapi import APIRouter, Request, HTTPException, status, Query
 from fastapi import Depends
-from rdflib import Graph, Literal, URIRef, PROF, DCTERMS
 from starlette.responses import PlainTextResponse
 
-from prez.cache import (
-    endpoints_graph_cache,
-    profiles_graph_cache,
-    links_ids_graph_cache,
-)
 from prez.dependencies import get_repo, get_system_repo
-from prez.models.listing import ListingModel
-from prez.models.object_item import ObjectItem
-from prez.models.profiles_and_mediatypes import ProfilesMediatypesInfo
 from prez.queries.object import object_inbound_query, object_outbound_query
-from prez.reference_data.prez_ns import PREZ
-from prez.renderers.renderer import return_from_graph, return_profiles
 from prez.routers.identifier import get_iri_route
-from prez.services.curie_functions import get_curie_id_for_uri, get_uri_for_curie_id
-from prez.services.model_methods import get_classes
 from prez.services.objects import object_function
-from prez.sparql.methods import Repo
-from prez.sparql.objects_listings import (
-    get_endpoint_template_queries,
-    generate_relationship_query,
-    generate_item_construct,
-    generate_listing_construct,
-    generate_listing_count_construct,
-)
 
 router = APIRouter(tags=["Object"])
 
@@ -93,5 +69,7 @@ async def count_route(
 
 
 @router.get("/object", summary="Object", name="https://prez.dev/endpoint/object")
-async def object_route(request: Request, repo=Depends(get_repo), system_repo=Depends(get_system_repo)):
+async def object_route(
+    request: Request, repo=Depends(get_repo), system_repo=Depends(get_system_repo)
+):
     return await object_function(request, repo=repo, system_repo=system_repo)
diff --git a/prez/routers/profiles.py b/prez/routers/profiles.py
@@ -1,6 +1,6 @@
 from fastapi import APIRouter, Request, Depends
 
-from prez.dependencies import get_repo, get_system_repo
+from prez.dependencies import get_system_repo
 from prez.services.objects import object_function
 from prez.services.listings import listing_function
 
@@ -44,4 +44,6 @@ async def profiles(
     name="https://prez.dev/endpoint/profile",
 )
 async def profile(request: Request, profile_curie: str, repo=Depends(get_system_repo)):
-    return await object_function(request, object_curie=profile_curie, repo=repo, system_repo=repo)
+    return await object_function(
+        request, object_curie=profile_curie, repo=repo, system_repo=repo
+    )