Skip to content

Conversation

@michaelmckinsey1
Copy link
Collaborator

@michaelmckinsey1 michaelmckinsey1 commented Aug 12, 2024

Working on RAJAPerf#471, I realized the string_attributes argument in GraphFrame.from_caliperreader was not accessible from the Thicket reader. This allows the user to do th.Thicket.from_caliperreader(file, string_attributes=[...]) in Thicket.

Edit: This is for reading complexity information from RAJAPerf runs.

@michaelmckinsey1 michaelmckinsey1 added area-thicket Issues and PRs involving Thicket's core Thicket datastructure and associated classes priority-normal Normal priority issues and PRs status-ready-for-review This PR is ready to be reviewed by assigned reviewers type-bug Identifies bugs in issues and identifies bug fixes in PRs labels Aug 12, 2024
@michaelmckinsey1 michaelmckinsey1 self-assigned this Aug 12, 2024
Copy link
Collaborator

@ilumsden ilumsden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@michaelmckinsey1 michaelmckinsey1 added status-approved No more revisions are required on this PR and it is ready for merge and removed status-ready-for-review This PR is ready to be reviewed by assigned reviewers labels Sep 3, 2024
@michaelmckinsey1 michaelmckinsey1 added this to the 2024.2.0 milestone Sep 4, 2024
@michaelmckinsey1 michaelmckinsey1 changed the title Use Kwargs in from_caliperreader Use Kwargs in from_caliperreader for RAJAPerf Complexities Sep 9, 2024
@slabasan slabasan merged commit a8387ce into llnl:develop Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-thicket Issues and PRs involving Thicket's core Thicket datastructure and associated classes priority-normal Normal priority issues and PRs status-approved No more revisions are required on this PR and it is ready for merge type-bug Identifies bugs in issues and identifies bug fixes in PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants