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.

Sign up for GitHub

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

Jump to bottom

Running Thicket.concat_thickets with calltree="intersection" produces empty query #199

Open
ilumsden opened this issue Jul 28, 2024 · 1 comment
Open

Running Thicket.concat_thickets with calltree="intersection" produces empty query #199

ilumsden opened this issue Jul 28, 2024 · 1 comment
Labels
area-thicket Issues and PRs involving Thicket's core Thicket datastructure and associated classes priority-normal Normal priority issues and PRs type-bug Identifies bugs in issues and identifies bug fixes in PRs

Comments

@ilumsden
Copy link
Collaborator

When trying to merge 4 Thickets with RAJAPerf data with concat_thickets, Thicket errors with EmptyQuery because the query used in Thicket.intersection would produce an empty Thicket object. After reviewing the individual Thickets by hand, I found that their call trees are identical, so there is a bug in Thicket.intersection causing this error.
@ilumsden ilumsden added area-thicket Issues and PRs involving Thicket's core Thicket datastructure and associated classes priority-normal Normal priority issues and PRs type-bug Identifies bugs in issues and identifies bug fixes in PRs labels Jul 28, 2024
@michaelmckinsey1
Copy link
Collaborator

May be fixed by the changes to intersection in #182

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-thicket Issues and PRs involving Thicket's core Thicket datastructure and associated classes priority-normal Normal priority issues and PRs type-bug Identifies bugs in issues and identifies bug fixes in PRs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ilumsden @michaelmckinsey1