Skip to content
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

Underscore prop _interpretation returning null when using batch import #1175

Closed
bobvanluijt opened this issue Jun 19, 2020 · 6 comments
Closed
Assignees
Labels

Comments

@bobvanluijt
Copy link
Member

bobvanluijt commented Jun 19, 2020

EDIT @etiennedi:
The issue seems to be limited to when the batch importer is used, but doesn't occur on the regular imports.


Original Issue:

To reproduce:

{
  Get {
    Things {
      Article {
        _interpretation {
          source {
            concept
            occurrence
            weight
          }
        }
        summary
      }
    }
  }
}

Tested both locally and on the WCS.

@bobvanluijt
Copy link
Member Author

note: _classification works correctly

@etiennedi etiennedi self-assigned this Jun 19, 2020
@etiennedi
Copy link
Member

Thanks for the bug ticket, investigating now. We do have a passing test, so my first suspicion would have to be that it's related to a configurable parameter (such as c11y version) where the test assumes one setting (instead of all).

@etiennedi
Copy link
Member

etiennedi commented Jun 19, 2020

Update: It works with the 0.14.0 contextionary, so something about the 0.16.0 must be broken. Since it's the same code base I can only imagine that something in the release process went wrong.

EDIT: This is not correct. Still searching for the cause.

@etiennedi
Copy link
Member

Update: It seems storing the _interpretation info is currently missing for the batch endpoints. It only works on the individual ones. I'll add that make a new release shortly.

@etiennedi etiennedi changed the title Underscore prop _interpretation returning null Underscore prop _interpretation returning null when using batch import Jun 19, 2020
@etiennedi
Copy link
Member

fixed in 0.22.10

@etiennedi
Copy link
Member

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants