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

FindDescriptor with Link Block Errors out #206

Closed
ifadams opened this issue Aug 9, 2024 · 1 comment · Fixed by #215
Closed

FindDescriptor with Link Block Errors out #206

ifadams opened this issue Aug 9, 2024 · 1 comment · Fixed by #215
Assignees
Labels
Bug Indicates unexpected or undesired behaviors
Milestone

Comments

@ifadams
Copy link
Contributor

ifadams commented Aug 9, 2024

Describe the bug
A FindDescriptor call with an included link block returns an error about non-unique returns

To Reproduce

Add a descriptor with a valid link to an existing metadata entity.

Then issue a find descriptor call, using a combination of FindEntity and Link, and use the affiliated link block with the FindDescriptor call, for example (in python)

    constraints = {"name": ["==", "Jane Doe"]}
    find_entity_q = query_helpers.find_entity("person", constraints=constraints, ref_tag=1234)

    link = query_helpers.create_link(1234)

    desc_find = {}
    desc_find["set"] = "pretty_faces"
    constraints = {"linked_prop": ["==",50]}
    desc_find["results"] = {"list": ["linked_prop"]}
    desc_find["link"] = link
    desc_find["constraints"] = constraints

    query_desc = {}
    query_desc["FindDescriptor"] = desc_find

    trans.append(find_entity_q)
    trans.append(query_desc)

Expected behavior

We should have the relevant descriptors linked to the specified FindEntity and their properties returned.

@ifadams ifadams added Bug Indicates unexpected or undesired behaviors In-Progress Tags issues that are actively being worked on. labels Aug 9, 2024
@ifadams ifadams self-assigned this Aug 9, 2024
@ifadams
Copy link
Contributor Author

ifadams commented Aug 9, 2024

The source of this error appears to be an accidental setting of the unique flag when querying against the descriptor set internally.

@cwlacewe cwlacewe added this to the v2.10.0 Tasks milestone Sep 10, 2024
@ifadams ifadams linked a pull request Sep 11, 2024 that will close this issue
@cwlacewe cwlacewe removed the In-Progress Tags issues that are actively being worked on. label Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Indicates unexpected or undesired behaviors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants