h5rdmtoolbox 2.5.3__tar.gz → 2.5.4__tar.gz

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: h5rdmtoolbox
-Version: 2.5.3
+Version: 2.5.4
 Summary: Supporting a FAIR Research Data lifecycle using Python and HDF5.
 Home-page: https://h5rdmtoolbox.readthedocs.io/en/latest/
 Author: Matthias Probst
@@ -29,7 +29,7 @@ Requires-Dist: pint>=0.24.4
 Requires-Dist: pint_xarray<=0.6.0,>=0.2.1
 Requires-Dist: regex>=2020.7.9
 Requires-Dist: packaging>=24.1
-Requires-Dist: ontolutils<1.0.0,>=0.22.1
+Requires-Dist: ontolutils<1.0.0,>=0.25.1
 Requires-Dist: python-forge==18.6.0
 Requires-Dist: requests>=2.32.4
 Requires-Dist: pydantic~=2.8
@@ -54,7 +54,7 @@ Provides-Extra: test
 Requires-Dist: pytest>=8.3.3; extra == "test"
 Requires-Dist: pytest-cov>=5.0.0; extra == "test"
 Requires-Dist: pylint; extra == "test"
-Requires-Dist: ssnolib<3.0.0,>=2.2.0.0; extra == "test"
+Requires-Dist: ssnolib<3.0.0,>=2.2.0.2; extra == "test"
 Requires-Dist: mongomock==4.1.2; extra == "test"
 Requires-Dist: xmltodict<=0.13.0; extra == "test"
 Requires-Dist: scipy>=1.10.1; extra == "test"
@@ -76,7 +76,7 @@ Requires-Dist: pymongo<=4.10.1,>=4.2.0; extra == "docs"
 Requires-Dist: pytest>=8.3.3; extra == "docs"
 Requires-Dist: pytest-cov>=5.0.0; extra == "docs"
 Requires-Dist: pylint; extra == "docs"
-Requires-Dist: ssnolib<3.0.0,>=2.2.0.0; extra == "docs"
+Requires-Dist: ssnolib<3.0.0,>=2.2.0.2; extra == "docs"
 Requires-Dist: mongomock==4.1.2; extra == "docs"
 Requires-Dist: xmltodict<=0.13.0; extra == "docs"
 Requires-Dist: scipy>=1.10.1; extra == "docs"
@@ -102,7 +102,7 @@ Requires-Dist: PyQt5==5.15.10; extra == "complete"
 Requires-Dist: pytest>=8.3.3; extra == "complete"
 Requires-Dist: pytest-cov>=5.0.0; extra == "complete"
 Requires-Dist: pylint; extra == "complete"
-Requires-Dist: ssnolib<3.0.0,>=2.2.0.0; extra == "complete"
+Requires-Dist: ssnolib<3.0.0,>=2.2.0.2; extra == "complete"
 Requires-Dist: mongomock==4.1.2; extra == "complete"
 Requires-Dist: xmltodict<=0.13.0; extra == "complete"
 Requires-Dist: scipy>=1.10.1; extra == "complete"
@@ -120,7 +120,7 @@ Requires-Dist: PyQt5==5.15.10; extra == "complete-with-docs"
 Requires-Dist: pytest>=8.3.3; extra == "complete-with-docs"
 Requires-Dist: pytest-cov>=5.0.0; extra == "complete-with-docs"
 Requires-Dist: pylint; extra == "complete-with-docs"
-Requires-Dist: ssnolib<3.0.0,>=2.2.0.0; extra == "complete-with-docs"
+Requires-Dist: ssnolib<3.0.0,>=2.2.0.2; extra == "complete-with-docs"
 Requires-Dist: mongomock==4.1.2; extra == "complete-with-docs"
 Requires-Dist: xmltodict<=0.13.0; extra == "complete-with-docs"
 Requires-Dist: scipy>=1.10.1; extra == "complete-with-docs"
@@ -141,7 +141,7 @@ Requires-Dist: pymongo<=4.10.1,>=4.2.0; extra == "complete-with-docs"
 Requires-Dist: pytest>=8.3.3; extra == "complete-with-docs"
 Requires-Dist: pytest-cov>=5.0.0; extra == "complete-with-docs"
 Requires-Dist: pylint; extra == "complete-with-docs"
-Requires-Dist: ssnolib<3.0.0,>=2.2.0.0; extra == "complete-with-docs"
+Requires-Dist: ssnolib<3.0.0,>=2.2.0.2; extra == "complete-with-docs"
 Requires-Dist: mongomock==4.1.2; extra == "complete-with-docs"
 Requires-Dist: xmltodict<=0.13.0; extra == "complete-with-docs"
 Requires-Dist: scipy>=1.10.1; extra == "complete-with-docs"

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/convention/core.py

@@ -702,7 +702,21 @@ def from_repo(repo_interface: RepositoryInterface,
         Name of the file to download
     """
     logger.debug(f"Downloading file {name} from repository {repo_interface}")
+
     filename = repo_interface.download_file(name)
+    _suffix = pathlib.Path(name).suffix
+    vfunc_filename = name.split(_suffix, 1)
+    has_vfunc = False
+    if len(vfunc_filename) == 2:
+        vfunc_filename = f'{vfunc_filename[0]}_vfuncs.py'
+        try:
+            downloaded_vfunc_filename = repo_interface.download_file(vfunc_filename)
+            has_vfunc = True
+        except Exception as e:
+            logger.debug(f'No vfuncs file found for {name}: {e}')
+    if has_vfunc:
+        shutil.copy(downloaded_vfunc_filename,
+                    filename.parent / downloaded_vfunc_filename.name)
     logger.debug(f"File downloaded to {filename}. Now loading convention from file.")
     return from_file(filename)
 

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/gui/fairify.py

@@ -10,7 +10,7 @@ import rdflib
 from PyQt5 import QtWidgets
 from PyQt5.QtGui import QFont
 from ontolutils import M4I, SCHEMA, OBO, CODEMETA, QUDT_UNIT, QUDT_KIND
-from ontolutils.classes.utils import split_URIRef
+from ontolutils.classes.utils import split_uri
 from pivmetalib.namespace import PIVMETA
 from ssnolib.namespace import SSNO
 
@@ -204,7 +204,7 @@ class Ui(QtWidgets.QMainWindow, Ui_MainWindow):
         if pred is None:
             pred = self.lineEditIRI1.text()
         self.log_message(f'updating prefix and suffix from text: {pred}')
-        ns, key = split_URIRef(pred)
+        ns, key = split_uri(pred)
         if ns is not None:
             if ns.startswith('http'):
                 for name, n in self.namespacelib.items():
@@ -221,7 +221,7 @@ class Ui(QtWidgets.QMainWindow, Ui_MainWindow):
         if pred is None:
             pred = self.lineEditIRI2.text()
         self.log_message(f'updating prefix and suffix from text: {pred}')
-        ns, key = split_URIRef(pred)
+        ns, key = split_uri(pred)
         self.log_message(f'Try to update combo boxes with ns={ns} and key={key}')
         if ns is not None:
             if ns.startswith('http'):
@@ -243,7 +243,7 @@ class Ui(QtWidgets.QMainWindow, Ui_MainWindow):
 
         self.log_message(f'Updating combo box with data: {current_prefix}...')
         self.comboBoxSuffix1.addItems(self.suffix_dict[current_prefix])
-        ns, key = split_URIRef(self.lineEditIRI1.text())
+        ns, key = split_uri(self.lineEditIRI1.text())
 
         ns_uri = self.namespacelib[current_prefix]._NS
 
@@ -258,7 +258,7 @@ class Ui(QtWidgets.QMainWindow, Ui_MainWindow):
 
         self.log_message(f'Updating combo box with data: {current_prefix}...')
         self.comboBoxSuffix2.addItems(self.suffix_dict[current_prefix])
-        ns, key = split_URIRef(self.lineEditIRI2.text())
+        ns, key = split_uri(self.lineEditIRI2.text())
 
         ns_uri = self.namespacelib[current_prefix]._NS
 

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/ld/user/groups.py

@@ -17,6 +17,7 @@ def process_group(group, graph, blank_node_iri_base: Optional[str] = None):
 
     group_uri = get_obj_bnode(group, blank_node_iri_base)
     rdf_manager = RDFManager(group.attrs)
+    parent_rdf_manager = RDFManager(group.parent.attrs) if group.parent is not None else None
     rdf_type = rdf_manager.type
     rdf_subject = rdf_manager.subject
 
@@ -27,6 +28,8 @@ def process_group(group, graph, blank_node_iri_base: Optional[str] = None):
         rdf_predicate = rdf_manager.predicate
         if rdf_predicate.get("SELF", None):
             graph.add((group_uri, rdflib.URIRef(rdf_predicate["SELF"]), rdflib.URIRef(rdf_subject)))
+            if parent_rdf_manager.subject:
+                graph.add((rdflib.URIRef(parent_rdf_manager.subject), rdflib.URIRef(rdf_predicate["SELF"]), rdflib.URIRef(rdf_subject)))
         rdf_file_predicate = rdf_manager.file_predicate
         if rdf_file_predicate is not None:
             file_uri = get_file_bnode(group.file, blank_node_iri_base)

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/repository/utils.py

@@ -26,8 +26,8 @@ def download_file(file_url,
         return existing_filename
 
     if target_folder is None:
-        from ..user import USER_CACHE_DIR
-        target_folder = USER_CACHE_DIR
+        from ..user import CACHE_DIR
+        target_folder = CACHE_DIR
     else:
         logger.debug(f'A target folder was specified. Downloading file to this folder: {target_folder}')
         target_folder = pathlib.Path(target_folder)
@@ -53,7 +53,12 @@ def download_file(file_url,
             except ValueError:
                 raise ValueError(f"Unsupported checksum algorithm: {checksum_algorithm}")
 
-        response = requests.get(file_url, stream=True, params={'access_token': access_token})
+        try:
+            response = requests.get(file_url, stream=True)
+            if response.status_code == 403:
+                response = requests.get(file_url, stream=True, params={'access_token': access_token})
+        except requests.RequestException as e:
+            response = requests.get(file_url, stream=True, params={'access_token': access_token})
         response.raise_for_status()
 
         chunk_size = 1024  # Define chunk size for download
@@ -101,7 +106,11 @@ def download_file(file_url,
         #                      f'Expected {checksum_algorithm} checksum: {checksum}, '
         #                      f'but got: {file_checksum}')
     else:
-        r = requests.get(file_url, params={'access_token': access_token})
+        try:
+            r = requests.get(file_url)
+        except requests.RequestException as e:
+            r = requests.get(file_url, params={'access_token': access_token})
+
         r.raise_for_status()
 
         try:
@@ -116,7 +125,10 @@ def download_file(file_url,
         with open(target_filename, 'wb') as file:
             file.write(r.content)
         if links_content:
-            _content_response = requests.get(links_content, params={'access_token': access_token})
+            try:
+                _content_response = requests.get(links_content)
+            except requests.RequestException:
+                _content_response = requests.get(links_content, params={'access_token': access_token})
             if _content_response.ok:
                 with open(target_filename, 'wb') as file:
                     file.write(_content_response.content)

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/repository/zenodo/core.py

@@ -203,15 +203,26 @@ class ZenodoRecord(RepositoryInterface):
         """Get the access token for the Zenodo API. This is needed to upload files."""
         return get_api_token(sandbox=self.sandbox, env_var_name=self.env_name_for_token)
 
-    def _get(self):
+    def _get(self, authenticate: bool = False):
+        """Get the deposit metadata as JSON. Using authenticate=True enforces authentication."""
         url = f"{self.depositions_url}/{self.rec_id}"
-        access_token = self.access_token
-        r = requests.get(url, params={"access_token": access_token})
+
+        if authenticate:
+            access_token = self.access_token
+            r = requests.get(url, params={"access_token": access_token})
+            r.raise_for_status()
+            return r.json()
+
+        record_url = url.replace("deposit/depositions/", "records/")
+        r = requests.get(record_url)
+        if r.status_code == 404:
+            access_token = self.access_token
+            r = requests.get(url, params={"access_token": access_token})
         r.raise_for_status()
         return r.json()
 
-    def get_metadata(self) -> Dict:
-        return self._get()['metadata']
+    def get_metadata(self, authenticate=False) -> Dict:
+        return self._get(authenticate=authenticate)['metadata']
 
     def set_metadata(self, metadata: Union[Dict, Metadata]):
         """update the metadata of the deposit"""
@@ -247,8 +258,13 @@ class ZenodoRecord(RepositoryInterface):
 
     def exists(self) -> bool:
         """Check if the deposit exists on Zenodo. Note, that only published records are detected!"""
-        resp = requests.get(self.record_url, params={'access_token': self.access_token})
-        return resp.ok
+        url = f"{self.depositions_url}/{self.rec_id}"
+        record_url = url.replace("deposit/depositions/", "records/")
+        r = requests.get(record_url)
+        if r.status_code == 404:
+            access_token = self.access_token
+            r = requests.get(url, params={"access_token": access_token})
+        return r.ok
 
     def is_published(self) -> bool:
         """Check if the deposit is published."""
@@ -268,6 +284,19 @@ class ZenodoRecord(RepositoryInterface):
             return url
 
         def _parse(data: Dict):
+            filename = data.get("filename", None)
+            if filename is None:
+                return dict(
+                    download_url=_parse_download_url(data['links']['self'], data['key']),
+                    access_url=f"https://doi.org/{self.get_doi()}",
+                    name=data["key"],
+                    media_type=_get_media_type(data.get('key')),
+                    identifier=data.get('id', None),
+                    identifier_url=None,
+                    size=data.get('size', None),
+                    checksum=data.get('checksum').strip("md5:") if data.get('checksum') else None,
+                    checksum_algorithm=data.get('checksum_algorithm', "md5"),
+                )
             return dict(
                 download_url=_parse_download_url(data['links']['download'], data['filename']),
                 access_url=f"https://doi.org/{self.get_doi()}",
@@ -343,10 +372,11 @@ class ZenodoRecord(RepositoryInterface):
         if new_version_string is not None and increase_part is not None:
             raise ValueError('Only one of new_version_string or increase_part must be provided.')
         if increase_part is not None:
-            new_version_string = _bump_version(self.get_metadata()['version'], increase_part)
+            metadata = self._get(authenticate=True)["metadata"]
+            new_version_string = _bump_version(metadata['version'], increase_part)
 
         self.unlock()
-        jdata = self._get()
+        jdata = self._get(authenticate=True)
 
         curr_version = Version(jdata['metadata']['version'])
         new_version = Version(new_version_string)
@@ -364,7 +394,7 @@ class ZenodoRecord(RepositoryInterface):
 
         new_record = copy.deepcopy(self)
         new_record.rec_id = _id
-        current_metadata = new_record.get_metadata()
+        current_metadata = new_record.get_metadata(authenticate=True)
         current_metadata["version"] = str(new_version)
         new_record.set_metadata(current_metadata)
         return new_record
@@ -402,7 +432,7 @@ class ZenodoRecord(RepositoryInterface):
         # Fallback: try to refresh deposit data (may still work) to get id
         if rec_id is None:
             try:
-                jdata = self._get()
+                jdata = self._get(authenticate=True)
                 rec_id = jdata.get("id")
             except Exception:
                 rec_id = None
@@ -424,13 +454,15 @@ class ZenodoRecord(RepositoryInterface):
         if not md:
             raise ValueError("No metadata found in the record.")
 
-        license_ = md.get("license") or None
+        license_ = md.get("license", None)
+        if isinstance(license_, dict):
+            license_ = license_["id"]
         creators = md.get("creators") or []
         keywords = md.get("keywords") or []
         doi = md.get("doi")
         landing = record.get("links", {}).get("self")
 
-        publisher = record["metadata"].get("imprint_publisher", None)
+        publisher = md.get("imprint_publisher", None)
         if publisher:
             if publisher.lower() == "zenodo":
                 publisher = foaf.Organization(
@@ -465,16 +497,17 @@ class ZenodoRecord(RepositoryInterface):
             key = f.get("key") or f.get("filename") or "file"
             doi = doi or str(record.get("doi"))
             title = key
-            media_type = pathlib.Path(f.get("filename")).suffix
-            size = f.get("filesize")
+            media_type = pathlib.Path(key).suffix
+            size = f.get("filesize", f.get("size", None))
             links = f.get("links", {})
             access_url = links.get("self") or record.get("links", {}).get("files")
-            download_url = links.get("download")
+            download_url = links.get("download", links.get("self"))
             if record["submitted"]:
                 download_url = download_url.replace("/draft/files", "/files")
             checksum_value = f.get("checksum")
             if checksum_value:
-                checksum_value = f"md5:{checksum_value}"
+                if not checksum_value.startswith(("md5:", "sha1:", "sha256:", "sha512:")):
+                    checksum_value = f"md5:{checksum_value}"
             checksum_obj = _spdx_checksum_from_zenodo(checksum_value)
 
             dist = {
@@ -507,7 +540,7 @@ class ZenodoRecord(RepositoryInterface):
             "distribution": distributions,
         }
         dcat_dataset = dcat.Dataset.model_validate(
-            dataset
+            remove_none(dataset)
         )
         return dcat_dataset
 
@@ -516,7 +549,7 @@ class ZenodoRecord(RepositoryInterface):
 
     def discard(self):
         """Discard the latest action, e.g. creating a new version"""
-        jdata = self._get()
+        jdata = self._get(authenticate=True)
         r = requests.post(jdata['links']['discard'],
                           params={'access_token': self.access_token})
         r.raise_for_status()
@@ -532,7 +565,7 @@ class ZenodoRecord(RepositoryInterface):
         APIError
             If the record cannot be unlocked because permission is missing.
         """
-        edit_url = self._get()['links'].get('edit', None)
+        edit_url = self._get(authenticate=True)['links'].get('edit', None)
         if edit_url is None:
             raise APIError('Unable to unlock the record. Please check your permission of the Zenodo API Token.')
 
@@ -567,7 +600,7 @@ class ZenodoRecord(RepositoryInterface):
         #     r.raise_for_status()
 
         # https://developers.zenodo.org/?python#quickstart-upload
-        bucket_url = self._get()["links"]["bucket"]
+        bucket_url = self._get(authenticate=True)["links"]["bucket"]
         logger.debug(f'adding file "{filename}" to record "{self.rec_id}"')
         with open(filename, "rb") as fp:
             r = requests.put(f"{bucket_url}/{filename.name}",
@@ -589,8 +622,14 @@ class ZenodoRecord(RepositoryInterface):
         else:
             target_filename = pathlib.Path(target_filename)
 
-        export_url = f"{self._get()['links']['html']}/export/{fmt}"
+        _links = self._get()["links"]
+        _html_url = _links.get("html", _links.get("self_html"))
+        export_url = f"{_html_url}/export/{fmt}"
+
         r = requests.get(export_url)
+        if r.status_code == 404:
+            access_token = self.access_token
+            r = requests.get(export_url, params={"access_token": access_token})
         r.raise_for_status()
         with open(target_filename, 'wb') as f:
             f.write(r.content)
@@ -614,3 +653,11 @@ class ZenodoRecord(RepositoryInterface):
                                     'owl': 'http://www.w3.org/2002/07/owl#',
                                     'dctype': 'http://purl.org/dc/dcmitype/'
                                     })
+
+
+def remove_none(obj):
+    if isinstance(obj, dict):
+        return {k: remove_none(v) for k, v in obj.items() if v is not None}
+    if isinstance(obj, list):
+        return [remove_none(v) for v in obj if v is not None]
+    return obj

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/user.py

@@ -1,3 +1,4 @@
+import os
 import pathlib
 import shutil
 import time
@@ -11,10 +12,11 @@ from ._version import __version__
 
 USER_LOG_DIR = pathlib.Path(appdirs.user_log_dir('h5rdmtoolbox', version=__version__))
 USER_DATA_DIR = pathlib.Path(appdirs.user_data_dir('h5rdmtoolbox', version=__version__))
-USER_CACHE_DIR = pathlib.Path(appdirs.user_cache_dir('h5rdmtoolbox', version=__version__))
+CACHE_DIR = pathlib.Path(appdirs.user_cache_dir('h5rdmtoolbox'))
 USER_LOG_DIR.mkdir(parents=True, exist_ok=True)
 USER_DATA_DIR.mkdir(parents=True, exist_ok=True)
-USER_CACHE_DIR.mkdir(parents=True, exist_ok=True)
+CACHE_DIR.mkdir(parents=True, exist_ok=True)
+USER_CACHE_DIR = CACHE_DIR  # deprecated name
 
 _filecounter = count()
 _dircounter = count()
@@ -22,6 +24,18 @@ _dircounter = count()
 _now = time.time()
 
 
+def default_cache_dir(__version__: str) -> pathlib.Path:
+    # Your current approach
+    return USER_DATA_DIR / "cache"
+
+
+def cache_dir(__version__: str) -> pathlib.Path:
+    override = os.environ.get("H5RDMTOOLBOX_CACHE_DIR")
+    if override:
+        return pathlib.Path(override)
+    return default_cache_dir(__version__)
+
+
 class DirManger:
     """Directory Manager class"""
 

{h5rdmtoolbox-2.5.3 → h5rdmtoolbox-2.5.4}/h5rdmtoolbox/utils.py

@@ -6,6 +6,7 @@ import json
 import logging
 import os
 import pathlib
+import random
 import re
 import time
 import uuid
@@ -25,10 +26,11 @@ from . import get_config, get_ureg
 from . import user
 from ._version import __version__
 from .ld import rdf
-from .user import USER_CACHE_DIR, USER_DATA_DIR
+from .user import CACHE_DIR, USER_DATA_DIR
 
 logger = logging.getLogger('h5rdmtoolbox')
 DEFAULT_LOGGING_LEVEL = logging.INFO
+RETRY_STATUS = {429, 500, 502, 503, 504}
 
 
 def get_filesize(filename: Union[str, pathlib.Path]) -> int:
@@ -52,12 +54,44 @@ def has_internet_connection(timeout: int = 5) -> bool:
         return False
 
 
+def _request_with_backoff(method, url, session=None, max_retries=8, timeout=30, **kwargs):
+    r = None
+    s = session or requests.Session()
+    for attempt in range(max_retries + 1):
+        r = s.request(method, url, timeout=timeout, **kwargs)
+
+        if r.status_code not in RETRY_STATUS:
+            return r
+
+        # Prefer server guidance
+        retry_after = r.headers.get("Retry-After")
+        if retry_after is not None:
+            try:
+                sleep_s = float(retry_after)
+            except ValueError:
+                sleep_s = None
+        else:
+            sleep_s = None
+
+        if sleep_s is None:
+            # Exponential backoff with jitter (full jitter)
+            base = min(60.0, 0.5 * (2 ** attempt))
+            sleep_s = random.uniform(0, base)
+
+        if attempt == max_retries:
+            return r
+
+        time.sleep(sleep_s)
+
+    return r
+
+
 def _download_file(url,
                    known_hash,
                    target: Optional[pathlib.Path] = None,
                    params: Optional[Dict] = None) -> pathlib.Path:
     """Download a file from a URL and check its hash"""
-    response = requests.get(url, stream=True, params=params)
+    response = _request_with_backoff('GET', url, params=params, stream=True)
     if response.status_code == 200:
         content = response.content
 
@@ -560,7 +594,8 @@ def download_context(url_source: Union[HttpUrl, List[HttpUrl]], force_download:
             logger.debug(f'Downloading context file from {_url} to {context_file}')
             try:
                 with open(context_file, 'wb') as f:
-                    f.write(requests.get(_url).content)
+                    r = _request_with_backoff("GET", _url)
+                    f.write(r.content)
             except requests.RequestException:
                 raise RuntimeError(f'Failed to download context file from {_url}')
         filenames.append(str(context_file))
@@ -608,7 +643,7 @@ class DownloadFileManager:
         return cls._instance
 
     def __init__(self):
-        self.file_directory = USER_CACHE_DIR
+        self.file_directory = CACHE_DIR
         self.file_directory.mkdir(parents=True, exist_ok=True)
         self.registry: Dict[str, Dict[str, str]] = self.load_registry()
         atexit.register(self.save_registry)