-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Heap size should be an on-chain parameter #312
Comments
Issue Status: 1. Open 2. Started 3. Submitted 4. Done This issue now has a funding of 225.0 DAI (225.0 USD @ $1.0/DAI) attached to it.
|
Hey @gavofyork , I'd like to work with you on this bounty. I'd like to use it as an opportunity to learn more about the product. |
@gavofyork do you have time this week or next week to talk about this issue? |
@dapperAuteur Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!
Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days |
I'm going to stop work. I added myself to this issue with the hopes of helping and learning. I wasn't able to open up communication with the bounty funder to learn anything about it. I may not have made my intentions clear when I started work on this bounty. I'm going to stop work now. I'll continue to learn about Rust and hopefully I'll be better equiped to help on future issues. |
Hey - sorry for not replying - Github quoting in not an effective means to get my attention. If you do start work on this, I'd be happy to review a PR. |
I can take a look at this issue, but I've one question, as we have a CLI option to specify the max heap pages, when user specifies this option, should we ignore the "heapsize" on the chain? |
@guanqun We've paid out the bounty for this issue to you via tip. Please let us know if you have any issues claiming, and great work! |
Thanks @vs77bb I'm not exactly sure how to claim that... Will try to figure this out. |
…t-timeout Increase block request timeout
At present the heap size is hard-coded to 128x 64KB pages (8MB); it should instead be specified as an on-chain similar to
:code
. The key could be:heapsize
.The text was updated successfully, but these errors were encountered: