sim-eng-ii comments

Results 22 comments of


                                            sim-eng-ii

Change usages of index(t) to a more robust isclose()

Hello @haggaila I tried to make the changes to `plot_routines.py` to resolve this issue. Let me know if I need to make any corrections. Thank you

Changing usages of index(t) to a more robust isclose()

@haggaila, thank you for the message, I will work on this and update you. Thank you.

Changing usages of index(t) to a more robust isclose()

@haggaila I made changes to the logic, I welcome feedback, suggestions. Thank you. (I will add a docstring to the function shortly)

Changing usages of index(t) to a more robust isclose()

Hi @haggaila, I've made some changes to the code based on your feedback. I've addressed the issue regarding the function returning `None` and added proper handling to ensure that the...

Remove `qiskit-terra` dependency in favor of `qiskit`

@1ucian0 was this issue resolved? If not, I would like to have a go.

Support verbatim boxes in `from_ir`

Hello, I encountered some issues after realising that some of the functions I was importing were causing a circular import error. I am now working on integrating the necessary classes...

Support verbatim boxes in `from_ir`

Hello @speller26, I was able to get the code working locally with the test that I added. To do so, I made several adjustments to make sure changes both repositories...

Support verbatim boxes in `from_ir`

@speller26 I updated the PR as per what we discussed, let me know if the changes I made are suitable or I can make further revisions 👍

feature: `add_verbatim_marker()` method to `BraketProgramContext` class for updated `Circuit.from_ir`

I am currently completing the changes in my fork of [amazon-braket-default-simulator-python](https://github.com/amazon-braket/amazon-braket-default-simulator-python). I will then open a PR, which should be reviewed alongside the one above. Thank you. I have also...

feature: `add_verbatim_marker()` method to `BraketProgramContext` class for updated `Circuit.from_ir`

hello @rmshaffer I will fix the code format issues