Ryan Shaffer

Results 51 comments of Ryan Shaffer

> Hi @rmshaffer would have really enjoyed working on this but I cannot do that at the moment due to factors beyond my control. Although I am following this repo...

Closing due to inactivity. Please re-open this one (or make a new PR) if you'd like to continue working on this!

Closing due to inactivity. Please re-open this one (or make a new PR) if you'd like to continue working on this!

> Hi! I would like to work on this task as part of unitaryhack. Hi @njones93531, thanks for your interest! Feel free to create a PR with your changes and...

Reopening this issue, as PR #675 was reverted by PR #701. Additionally, I think there is still a need to document the fact that OpenQASM language-reserved words (such as `input`,...

> I don't see on #701 the reason for reverting. Anyone know what the issue was? I believe there was an issue with a particular OpenQASM parser not properly handling...

@Tarun-Kumar07 I believe we should do something more like your Solution 1, but only when FreeParameter matches one of the OpenQASM reserved words, or one of the SDK predefined variables...

I don't believe there would be any reason for the SDK to introduce new predefined variables beyond `b` and `q`. If so, this would need to be accounted for at...

Hi @Tarun-Kumar07 - could you please open a draft PR with your code so that we could more easily review your changes? I don't know what would cause those four...

@Tarun-Kumar07 PR #999 has been merged. Thank you for your contribution!