You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@avinxshKD I am already working on this issue. Thanks for offering your help but please work on different issue.
Just to clarify, I was adding suggestions to the discussion, not claiming the issue. All good, I’ll put up a PR for the test cases I mentioned. If thats okay @pradeeban, thanks
It is a dangerous task. There was a PR I closed: #213.
concore supports directed hypergraphs.
If you are simply validating whether a graphml is valid, it should be ok.
But assuming all the graphml files are produced by concore-editor, I believe we won't have invalid graphml files? Just be cautious with this fix. We do not want to unintentionally introduce bugs while trying to fix a problem that does not exist in practice (since this is handled by concore-editor, unless someone mistakenly tries to run a graphml not created by concore-editor, which is unlikely).
Activity
GREENRAT-K405 commentedon Feb 4, 2026
@pradeeban , what are your thoughts on this? is this required?
I'd like to work on this issue!
avinxshKD commentedon Feb 7, 2026
Makes sense ,, I'd suggest we test at least:
@pradeeban want me to draft a PR with a few test cases?
GREENRAT-K405 commentedon Feb 7, 2026
@avinxshKD I am already working on this issue. Thanks for offering your help but please work on different issue.
avinxshKD commentedon Feb 7, 2026
Just to clarify, I was adding suggestions to the discussion, not claiming the issue. All good, I’ll put up a PR for the test cases I mentioned. If thats okay @pradeeban, thanks
pradeeban commentedon Feb 8, 2026
It is a dangerous task. There was a PR I closed: #213.
concore supports directed hypergraphs.
If you are simply validating whether a graphml is valid, it should be ok.
But assuming all the graphml files are produced by concore-editor, I believe we won't have invalid graphml files? Just be cautious with this fix. We do not want to unintentionally introduce bugs while trying to fix a problem that does not exist in practice (since this is handled by concore-editor, unless someone mistakenly tries to run a graphml not created by concore-editor, which is unlikely).
pradeeban commentedon Feb 8, 2026
Closing this since #218 is merged.