Skip to content

Node map generation seems to miss anonymous graph name #144

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
gkellogg opened this issue Aug 31, 2019 · 1 comment · Fixed by #145
Closed

Node map generation seems to miss anonymous graph name #144

gkellogg opened this issue Aug 31, 2019 · 1 comment · Fixed by #145

Comments

@gkellogg
Copy link
Member

As noted in #143, the following inputs should create named graphs, but don't:

{
  "@context": {
    "@version": 1.1,
    "@vocab": "http://example.org/",
    "input": {"@container": "@graph"}
  },
  "input": [{
    "@id": "ex:someIdentifier",
    "@graph": [
      {"value": "x"},
      {"value": "y"}
    ]
  }]
}

and

{
  "@context": {
    "@version": 1.1,
    "@vocab": "http://example.org/",
    "input": {"@container": "@graph"}
  },
  "input": [{
    "@graph": [
      {"value": "x"},
      {"value": "y"}
    ]
  }]
}
@dlongley
Copy link
Contributor

I think this is perhaps just a symptom of the special case code we've been discussing in #143 as removing it addresses the problem.

gkellogg added a commit that referenced this issue Sep 1, 2019
…ers where the value is already a graph.

Fixes #144. Relates to #143.
gkellogg added a commit that referenced this issue Sep 2, 2019
…ers where the value is already a graph.

Fixes #144. Relates to #143.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants