This PR fixes the issue reported in Issue #16 where the summaries were stored as null in the index.store.json file when creating a ComposableGraph and persisting it.
Changes Made
Modified the from_indices method in ComposableGraph class to correctly assign the summaries to the indices when creating the graph.
Added a check in the persist method of StorageContext class to ensure that the summaries of the indices in the graph are not null before persisting the graph.
Testing
Created test cases to verify that the summaries are correctly assigned to the indices in the ComposableGraph.
Tested the persist method to ensure that it raises an error or warning if any summary is null.
Description
This PR fixes the issue reported in Issue #16 where the summaries were stored as null in the
index.store.json
file when creating aComposableGraph
and persisting it.Changes Made
from_indices
method inComposableGraph
class to correctly assign the summaries to the indices when creating the graph.persist
method ofStorageContext
class to ensure that the summaries of the indices in the graph are not null before persisting the graph.Testing
ComposableGraph
.persist
method to ensure that it raises an error or warning if any summary is null.Related Issue
Issue #16
Checklist
Screenshots (if applicable)
N/A
Fixes #16.
To checkout this PR branch, run the following command in your terminal: