Skip to content
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

Implement "render function timeouts" to prevent a specific render function e.g. doInitial* from running for more than a configurable 500ms #348

Open
seveibar opened this issue Nov 23, 2024 · 5 comments

Comments

@seveibar
Copy link
Contributor

seveibar commented Nov 23, 2024

Emit an error "<trace ... />" took more than 500ms to execute doInitialPcbComponentRender

/bounty $30

Copy link

algora-pbc bot commented Nov 23, 2024

💎 $30 bounty • tscircuit

Steps to solve:

  1. Start working: Comment /attempt #348 with your implementation plan
  2. Submit work: Create a pull request including /claim #348 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to tscircuit/core!

Add a bountyShare on socials

Attempt Started (GMT+0) Solution
🔴 @ShiboSoftwareDev Nov 24, 2024, 8:01:17 AM WIP

@ShiboSoftwareDev
Copy link
Contributor

ShiboSoftwareDev commented Nov 24, 2024

/attempt #348 I'll give it a shot

Algora profile Completed bounties Tech Active attempts Options
@ShiboSoftwareDev 16 tscircuit bounties
TypeScript, JavaScript,
MDX & more
﹟35, ﹟331
Cancel attempt

@seveibar
Copy link
Contributor Author

@ShiboSoftwareDev I've been looking into this one for a bit, I think it's basically impossible w/o webworkers, I probably should have investigated more before bountying. Lmk if you come to a different conclusion!!!

@ShiboSoftwareDev
Copy link
Contributor

@ShiboSoftwareDev I've been looking into this one for a bit, I think it's basically impossible w/o webworkers, I probably should have investigated more before bountying. Lmk if you come to a different conclusion!!!

@anas-sarkez

@seveibar
Copy link
Contributor Author

At this point I think this is mostly not possible (technically there is probably a solution but it's a dramatic architecture change), there is a version of this that is possible by considering the maximum execution time of effects, but imo we should basically do most of the freeze detection via webworkers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants