Skip to content

Commit

Permalink
python: drop defaults from JGF edges
Browse files Browse the repository at this point in the history
Problem: the JGF reader does not check the 'directed' field of edges
and edges are assumed to be containment by default. Some space could
be saved by dropping those two fields.

Drop them.
  • Loading branch information
jameshcorbett committed Sep 18, 2024
1 parent 86ed74e commit 846a327
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/python/fluxion/resourcegraph/V1.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ def __init__(self, parentId, vtxId):
super().__init__(
parentId,
vtxId,
directed=True,
metadata={"subsystem": "containment"},
)


Expand All @@ -105,7 +103,8 @@ def __init__(self, rv1):
rv1 -- RV1 Dictorary that conforms to Flux RFC 20:
Resource Set Specification Version 1
"""
super().__init__()
super().__init__(directed=False)
# graph *is* directed, however, suppress unnecessary `"directed": true` fields
self._uniqId = 0
self._rv1NoSched = rv1
self._encode()
Expand Down

0 comments on commit 846a327

Please sign in to comment.