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

dialects: (measurement) add dialect and attributes for measurement #41

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

alexarice
Copy link
Collaborator

Adds attributes for measurement similar to the attributes for allocation and gates.

@alexarice alexarice added dialects Changes on the dialects core labels Jan 28, 2025
@alexarice alexarice requested a review from kimxworrall January 28, 2025 16:57
@alexarice alexarice self-assigned this Jan 28, 2025
Copy link
Contributor

@kimxworrall kimxworrall left a comment

Choose a reason for hiding this comment

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

Looks good. Probably a bit pedantic, but worth testing measurexyattr

@alexarice alexarice requested a review from kimxworrall January 31, 2025 14:43
@alexarice alexarice changed the title dialects: (measure) add dialect and attributes for measurement dialects: (measurement) add dialect and attributes for measurement Jan 31, 2025
@alexarice
Copy link
Collaborator Author

Don't think I renamed the test files...

@alexarice
Copy link
Collaborator Author

Nevermind I didn't have any on this PR

@alexarice
Copy link
Collaborator Author

MeasureXYAttr is tested as part of qssa. Has its own test added in the next PR

@alexarice alexarice force-pushed the alexarice/measure-attrs branch from 3269d32 to 95f9c8b Compare January 31, 2025 16:13
@alexarice alexarice force-pushed the alexarice/measure-attrs branch from 95f9c8b to 2ca7470 Compare January 31, 2025 16:23
@alexarice
Copy link
Collaborator Author

I'm going to start merging, we can iterate on names later

@alexarice alexarice merged commit 4ff3af6 into main Feb 3, 2025
5 checks passed
@alexarice alexarice deleted the alexarice/measure-attrs branch February 3, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core dialects Changes on the dialects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants