-
Notifications
You must be signed in to change notification settings - Fork 248
[Testing] Tests might saturate MyBinder requests #751
Comments
@Tansito, @vabarbosa, do you consider this to be a bug? |
Mmm... Maybe yes, but definitely this is something that we should increase the priority and take a look whether it's a bug or not I think. |
this could be a high priority issue since it is can possibly impact users but i am havent heard of any issues, yet. |
Hi, https://mybinder.org/ is receiving a huge traffic related to https://github.com/Qiskit/platypus-binder. At the moment, https://github.com/Qiskit/platypus-binder is driving more than half of the traffic in https://mybinder.org/ https://mybinder.org/ is running with reduced capacity since May 2023 (details at https://blog.jupyter.org/mybinder-org-reducing-capacity-c93ccfc6413f) and providing CPU time for Qiskit Textbook (
Will be possible to use Pyodide to run the code exercises for the textbook? |
thank you @rgaiacs we are investigating |
@rgaiacs we've updated our application to (hopefully) significantly minimize the load on mybinder. |
@vabarbosa Thanks for the pull request. I will follow up tomorrow. |
@vabarbosa Thanks again for the pull request! The traffic on mybinder.org from https://github.com/Qiskit/platypus-binder dropped to 1/4 after #2132 was merged. ❤️ |
This repository is no longer maintained. It has been superseded by IBM Quantum Learning. |
I am not sure if this is a real problem, but I suspect it is. We have to take a look at it.
MyBinder is the service we are using to execute the python CodeCells remotely. It has a limited number of requests per hour that we can do.
When a page loads, it makes a request to connect with MyBinder and be ready to execute any code.
During the tests, there will be many requests visiting every page. Cypress clears the cookies very often so that it won't take advantage of the cached connection.
A possible solution is to put a cookie or localStorage key to prevent requests during testing.
The text was updated successfully, but these errors were encountered: