Skip to content

Migrate to the NamedGraphs v0.14 interface - #180

Merged
mtfishman merged 3 commits into
release-0.10from
mf/namedgraphs-v0.14-release-0.10
Sep 9, 2026
Merged

mtfishman merged 3 commits into
release-0.10from
mf/namedgraphs-v0.14-release-0.10

Conversation

@mtfishman

@mtfishman mtfishman commented Sep 9, 2026

Copy link
Copy Markdown
Member

Summary

Migrates the 0.9 line to NamedGraphs v0.14, which removed the submodules and the position-graph interface this package imported. The overloads move to the encode and decode interface, and the mutating ones now return Bool as Graphs.jl documents. This is #174 ported to release-0.10, released as 0.10.0.

NamedGraphs v0.14 removed the submodules and the position-graph interface this package imported, so it no longer loads. The overloads move to the encode and decode interface, and the dead imports go with them, since an import of a removed name fails at load rather than merely being noise.
Graphs.jl documents these as returning whether the operation happened, and DataGraphs now does too, so returning the network or cache left this package inconsistent with its own base class. Also fixes `add_vertex!` on a `MessageCache`, which called `add_edge!` and so could never succeed.
@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 72.22222% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.79%. Comparing base (c90d395) to head (18d3c82).

Files with missing lines Patch % Lines
src/normnetworkview.jl 0.00% 5 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release-0.10     #180      +/-   ##
================================================
+ Coverage         84.81%   85.79%   +0.98%     
================================================
  Files                15       15              
  Lines               685      683       -2     
================================================
+ Hits                581      586       +5     
+ Misses              104       97       -7     
Flag Coverage Δ
docs 0.00% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mtfishman mtfishman changed the title [WIP] Migrate to the NamedGraphs v0.14 interface Migrate to the NamedGraphs v0.14 interface Sep 9, 2026
@mtfishman
mtfishman marked this pull request as ready for review September 9, 2026 13:25
@mtfishman
mtfishman merged commit 4cfaf02 into release-0.10 Sep 9, 2026
23 of 26 checks passed
@mtfishman
mtfishman deleted the mf/namedgraphs-v0.14-release-0.10 branch September 9, 2026 16:05
mtfishman added a commit that referenced this pull request Sep 9, 2026
## Summary

Moves `main` to `0.11.0-DEV`. The NamedGraphs v0.14 migration ships as
0.10.0 from `release-0.10`
(#180), so the
breaking changes accumulating here release as 0.11.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant