Repository navigation
Add Qiskit Circuit Resource, Runtime & Cost Estimator - #1439
qiskit-bot wants to merge 2 commits into
Conversation
… a member Co-authored-by: troncos22 <troncos22@users.noreply.github.com>
Luciano Bello (1ucian0)
left a comment
Luciano Bello (1ucian0)
left a comment
There was a problem hiding this comment.
Thanks for the submission!
The project looks a bit... messy...?
The README file feels half organized/written. It points to a unit test file that isn’t in the repository. The title Idea 1 makes little sense to me.
qiskit_circuit_estimator.py never imports qiskit or qiskit_aer.
GitHub license detection is not working neither.
The hardware references are out of date. Take a look to IBM’s retired QPU list
The committed __pycache__/*.pyc should be removed.
Just to mention few :)
| @@ -0,0 +1,11 @@ | |||
| name = "Qiskit Circuit Resource, Runtime & Cost Estimator" | |||
There was a problem hiding this comment.
Why not Qtester? (I dont know what mf stands for :)
| url = "https://github.com/troncos22/Qtester_mf.git" | ||
| description = "Resource analyzer and runtime cost estimator for Qiskit circuits on IBM Quantum hardware." | ||
| contact_info = "jorgmoreno@alumnos.uai.cl" | ||
| labels = ["benchmarking", "optimization", "demo"] |
There was a problem hiding this comment.
The code neither benchmarks software or hardware nor related to quantum optimization. demo is plausible but also not fully sure. Take a look to https://qisk.it/ecosystem-labels
There was a problem hiding this comment.
hi i re-desing the code, does the same, but in a diferent way y already make the push, how can i "re-aply" to make a push?
There was a problem hiding this comment.
you can mention the changes here and we can modify this very own application.
There was a problem hiding this comment.
Hi Luciano, thanks for the help!
I am attaching the redesigned files here for you to update the application. Here is a summary of the changes made to address the feedback:
qiskit_circuit_estimator.py: Completely restructured the main logic. It now properly imports qiskit and qiskit_aer, includes precision selection (single/double), and automatically detects RAM/VRAM for both CPU and GPU execution.
actualizacion_data_QPU.py: Added a new web scraping script (using Playwright) to pull the active IBM QPU list dynamically, ensuring we no longer reference retired hardware.
README.md: Rewritten to be much clearer, removing the confusing reference to the non-existent unit test and explaining how to use the new scripts.
requirements.txt: Added to explicitly list dependencies (including qiskit-aer-gpu and playwright).
LICENSE: Added a standard MIT License file so GitHub's license detection works correctly.
Cleanup: Removed the pycache directory and all .pyc files from the tracking.
(Note: You can attach the files directly to this comment by dragging and dropping them into the text box, or paste a link to your own repository branch if you have them hosted there).
Let me know if you need me to format this differently or if there is anything else I should adjust!
Co-authored-by: Luciano Bello <766693+1ucian0@users.noreply.github.com>
Nomination for: Qiskit Circuit Resource, Runtime & Cost Estimator
By Jorge (@troncos22), via #1438