Release 0.13.5

Author: apdavison

codemeta.json

@@ -4,15 +4,15 @@
   "license": "https://spdx.org/licenses/Apache-2.0.html",
   "codeRepository": "https://github.com/HumanBrainProject/fairgraph",
   "contIntegration": "https://github.com/HumanBrainProject/fairgraph/actions",
-  "dateModified": "2026-05-14",
-  "downloadUrl": "https://files.pythonhosted.org/packages/d7/f6/460a038f133fd922eac20d544a14fe5c7b1a214ce9f02c9f53d2392e9291/fairgraph-0.13.4.tar.gz",
+  "dateModified": "2026-05-26",
+  "downloadUrl": null,
   "issueTracker": "https://github.com/HumanBrainProject/fairgraph/issues",
   "name": "fairgraph",
-  "version": "0.13.4",
-  "identifier": "https://pypi.org/project/fairgraph/0.13.4/",
+  "version": "0.13.5",
+  "identifier": "https://pypi.org/project/fairgraph/0.13.5/",
   "description": "Python API for the EBRAINS Knowledge Graph",
   "applicationCategory": "neuroscience",
-  "releaseNotes": "https://fairgraph.readthedocs.io/en/latest/release_notes.html#version-0-13-4",
+  "releaseNotes": "https://fairgraph.readthedocs.io/en/latest/release_notes.html#version-0-13-5",
   "funding": "https://cordis.europa.eu/project/id/945539",
   "developmentStatus": "active",
   "referencePublication": null,

doc/release_notes.rst

@@ -3,6 +3,19 @@ Release notes
 =============
 
 
+Version 0.13.5
+==============
+
+Bug fixes in this release:
+
+- Fixed a bug where :meth:`~fairgraph.kgobject.KGObject.save` could silently
+  no-op after re-fetching an object via :meth:`~fairgraph.kgobject.KGObject.from_id`,
+  because :attr:`KGClient.cache` was not invalidated by writes
+  (`#110 <https://github.com/HumanBrainProject/fairgraph/issues/110>`_).
+- Fixed the repository IRI used by the dataset version ``download()`` method
+  (`#109 <https://github.com/HumanBrainProject/fairgraph/pull/109>`_).
+
+
 Version 0.13.4
 ==============
 

fairgraph/__init__.py

@@ -28,7 +28,7 @@
 from .collection import Collection
 from . import client, errors, openminds, utility
 
-__version__ = "0.13.4"
+__version__ = "0.13.5"
 
 utility.initialise_instances(
     [

fairgraph/kgobject.py

@@ -763,6 +763,10 @@ def save(
                             raise
                     else:
                         self.remote_data = local_data
+                        # Mark the raw server-side document as stale, to match
+                        # the cache invalidation done in client.replace_instance.
+                        # It will be re-fetched on demand by exists() if needed.
+                        self._raw_remote_data = None
                 else:
                     modified_data = self.modified_data()
                     if modified_data:
@@ -792,6 +796,11 @@ def save(
                                     raise
                             else:
                                 self.remote_data = local_data
+                                # Mark the raw server-side document as stale, to
+                                # match the cache invalidation done in
+                                # client.update_instance. It will be re-fetched
+                                # on demand by exists() if needed.
+                                self._raw_remote_data = None
                             if activity_log:
                                 activity_log.update(
                                     item=self,

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "fairgraph"
-version = "0.13.4"
+version = "0.13.5"
 description = "Python API for the EBRAINS Knowledge Graph"
 readme = "README.md"
 authors = [

test/test_client.py

@@ -346,3 +346,45 @@ def contribute_to_partial_replacement(instance_id, payload, extended_response_co
         assert studied_specimen_path not in server_state, (
             "second save should have sent a PATCH that cleared studiedSpecimen"
         )
+
+    def test_save_marks_raw_remote_data_stale(self, offline_kg_client, mocker):
+        """After a successful update/replace, `_raw_remote_data` must be set
+        to None so it can't be silently out of sync with the cache and with
+        the actual server state. exists() repopulates it on demand."""
+        from fairgraph.openminds.core import DatasetVersion
+
+        server_state = {
+            "@id": self.uri,
+            "@type": ["https://openminds.om-i.org/types/DatasetVersion"],
+            "http://schema.org/identifier": [self.uri],
+            "https://core.kg.ebrains.eu/vocab/meta/space": "myspace",
+            "https://openminds.om-i.org/props/shortName": "original",
+        }
+
+        def get_by_id(stage, instance_id, extended_response_configuration):
+            return MockKGResponse(dict(server_state))
+
+        def contribute_to_partial_replacement(instance_id, payload, extended_response_configuration):
+            for key, value in payload.items():
+                if value is None:
+                    server_state.pop(key, None)
+                else:
+                    server_state[key] = value
+            return MockKGResponse(dict(server_state))
+
+        mocker.patch.object(offline_kg_client._kg_client.instances, "get_by_id", get_by_id)
+        mocker.patch.object(
+            offline_kg_client._kg_client.instances,
+            "contribute_to_partial_replacement",
+            contribute_to_partial_replacement,
+        )
+
+        dsv = DatasetVersion.from_id(self.uuid, offline_kg_client, scope="any")
+        assert dsv._raw_remote_data is not None  # populated by from_id
+
+        dsv.short_name = "updated"
+        dsv.save(offline_kg_client, space="myspace", recursive=False)
+
+        assert dsv._raw_remote_data is None, (
+            "_raw_remote_data must be invalidated after a successful update"
+        )

test/test_openminds_core.py

@@ -884,12 +884,12 @@ def test_dataset_version_download(mocker):
         lambda url, local_filename: (local_filename, None),
     )
     fake_dsv = omcore.DatasetVersion(
-        repository=omcore.FileRepository(iri=IRI("https://data-proxy.ebrains.eu/api/v1/public/foo/bar"))
+        repository=omcore.FileRepository(iri=IRI("https://data-proxy.ebrains.eu/api/v1/buckets/foo/bar"))
     )
     local_dir = tempfile.mkdtemp()
     local_filename, repository_url = fake_dsv.download(local_dir, client=None, accept_terms_of_use=True)
     os.rmdir(local_dir)
-    assert repository_url == "https://data-proxy.ebrains.eu/api/v1/public/foo/bar"
+    assert repository_url == "https://data-proxy.ebrains.eu/api/v1/buckets/foo/bar"
     assert str(local_filename) == os.path.join(local_dir, "bar.zip")