-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Create visualization tests for pulse #6502
Comments
Dear @enavarro51 , I'd have a look at this, if possible. |
hey @MartinBeseda sorry for the slow response, are you still interested in working on this? If yes lmk and I'll assign to you 😄 |
Hey, @javabster can I work on this? |
Hi @ozamanan as @MartinBeseda is unresponsive I will assign to you, let us know if you have any questions 😄 |
Hi @javabster, Add text_pulse_visualization_output.txt in this folder |
Hi @anonymousr007 actually I think the point of this issue is to create some binder style snapshot tests for pulse, using the tests found in |
Any news @ozamanan ? |
Hi @anonymousr007 |
The expected enhancement is to create a set of image comparison tests for pulse, similar to the existing #4544 binder style tests. These tests will help ensure that the pulse images generated by different pulse programs are consistent and correct. The tests can be created based on the old pulse image comparison tests that were previously skipped, which were removed as part of #6467. By adding these tests back in a new form, the pulse functionality can be more thoroughly tested and validated @1ucian0 Can I Get assigned to this issue |
Thanks @AbdullahKazi500 ! assigning. Small headsup. The binder mechanism is growing intro proper CI testing in #9961 . There are two options at this point:
Up to you. Whatever you feel more confortable with. |
Thank you for the heads up! It's good to know that the binder mechanism is moving towards proper CI testing. I'll keep these options in mind when writing pulse tests. It seems like using the notebook way first and then moving to CI testing would be the safer option in case there are any changes in #9961. |
Hi @Raghav-Bell Yeah I am working on this for now |
Hi @AbdullahKazi500 How is that going? |
@1ucian0 Hi Luciano I have created a Jupyter Notebook-based tests for pulse that run as part of the Continuous Integration (CI) process. These notebooks should include the necessary test cases and comparisons. |
What is the expected enhancement?
As part of #6467, some old pulse image comparison tests that were being skipped were removed. These were in the file test_pulse_visualization_output.txt. The tests in this file may be useful when a #4544 binder style set of image comparison tests is created for pulse.
The text was updated successfully, but these errors were encountered: