Comments (29)
Cool I just saw that the VIP got approved. Congrats!
from casper.
@djrtwo Your proposed solution is syntactically elegant, but from what I can see, it's going to require an enhancement to Vyper. Vyper's RLPList
function sets the size of
all byte array members equal to the size of the entire RLPList.
For the purposes of decoding incoming data, Vyper's RLPList
requires as arguments a bytearray, and a serial list of types -- the latter is used by RLPList to decode the former into an array of native objects. It is used like this by Casper:
casper/casper/contracts/simple_casper.v.py
Line 491 in dcf4caf
The above doesn't expose any way to let Vyper's RLPList
know how many bytes will be in the byte array members, so (again, linked above) RLPList
just allocates a number of bytes equal to the size allocated for the entire message, in our case, 1024
.
To summarize, due to the current implementation of RLPList
in Vyper, the size of the variable sig
referenced below must be equal to the size of the entire message (vote_msg
) it's contained in.
casper/casper/contracts/simple_casper.v.py
Lines 486 to 496 in dcf4caf
I've looked into making some enhancements to the Vyper RLPList
implementation, but it's a fairly involved undertaking, and I'd need to understand more about related project pyrlp.
Requesting a suggestion on how I should proceed.
from casper.
Thanks for the deep dive! I was unaware. We might decide to truncate or something, but I am going to discuss with the formal verification team that is working with us. Leaving this open for now until we decide on a move forward.
from casper.
@mkeen @djrtwo Just 10-day-snoozed Gitcoin Bot, appreciate the request and patience as we work out the kinks 🙂
from casper.
@djrtwo Would it be better to add this enforcement in validate_signature()?
from casper.
The type declaration should handle the enforcement. I was thinking we just make the type declaration the following anywhere we are referencing the sig which is a number of places at this point (including validate_signature
)
sig: bytes <= MAX_SIGNATURE_LENGTH = ...
from casper.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
This issue now has a funding of 200.0 DAI (200.0 USD @ $1.0/DAI) attached to it.
- If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
- Questions? Checkout Gitcoin Help or the Gitcoin Slack
- $8,316.90 more funded OSS Work available on the Gitcoin Issue Explorer
from casper.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
Work has been started.
These users each claimed they can complete the work by 7 months, 2 weeks ago.
Please review their action plans below:
1) mkeen has started work.
Learn more on the Gitcoin Issue Details page.
from casper.
@mkeen 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!
- warning (3 days)
- escalation to mods (6 days)
Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days
from casper.
I am still working on this. Submitted a proposal to make this possible with a change to Vyper.
from casper.
@mkeen 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!
- warning (3 days)
- escalation to mods (6 days)
Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days
from casper.
@djrtwo Can I get a 10 day snooze on this? RLPList is undergoing heavy changes RN.
from casper.
@mkeen I can't snooze as I'm not the founder. will have to ask @gdipri01
from casper.
Thanks!
from casper.
Hi @mkeen just checking in, is this one still pending? Hope you are doing well 🙂
from casper.
This is blocked by the implementation of a VIP vyperlang/vyper#818
It appears to be in their "final countdown" before beta. So I expect it to be finished in the relative short term
from casper.
@mkeen 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!
- warning (3 days)
- escalation to mods (6 days)
Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days
from casper.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
@mkeen due to inactivity, we have escalated this issue to Gitcoin's moderation team. Let us know if you believe this has been done in error!
- warning (3 days)
- escalation to mods (6 days)
Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days
from casper.
Hi @vs77bb yes still pending! But I am keeping an eye on it.
from casper.
Awesome. Thanks for the feedback!
from casper.
Hi there everyone. Is this issue still blocked due to your dependencies?
Let me know if we can help you in any ways!
Thanks!
from casper.
@kuhnchris The repo is deprecated. No commits since Jun 2018.
from casper.
Well, there are still 200$ in DAI stashed in the bounty from Greg. If this repository is deprecated he should cancel the bounty so he can at least get his funds back. We'll try to mail him, thanks for the info!
May I note that if this is deprecated that maybe the repository should be changed to read-only, respectively archived?
Thanks!
Chris
from casper.
The repo isn't deprecated as far as I know. It would be nice to at least partially claim the bounty, since I did a bunch of analysis and spec'ed out a fix upstream. Just a thought :)
from casper.
@djrtwo @hrishikeshio your calls. cc @vs77bb
from casper.
@djrtwo @hrishikeshio @vs77bb LMK. I think it's only fair.
from casper.
Its deprecated according to https://github.com/ethereum/casper/blob/master/README.md.
Anyways, I am not the decision maker or bounty sponsorer. So its not my decision. good luck.
from casper.
Yikes.
from casper.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
Work for 200.0 DAI (200.0 USD @ $1.0/DAI) has been submitted by:
@gdipri01 please take a look at the submitted work:
- (Link Not Provided) by @mkeen
- Learn more on the Gitcoin Issue Details page
- Want to chip in? Add your own contribution here.
- Questions? Checkout Gitcoin Help or the Gitcoin Slack
- $86,212.92 more funded OSS Work available on the Gitcoin Issue Explorer
from casper.
Related Issues (20)
- ubuntu install issue HOT 4
- Can't call `init` and `initialize_epoch` at same block number HOT 1
- Contract warmup period HOT 1
- next_validator_index comment is misleading HOT 2
- `slash` when validator has deposited but not yet reached `start_dynasty` HOT 4
- `assert msg.sender == NULL_SENDER` in `vote` HOT 3
- Port testing to `eth-tester` HOT 2
- Exposing `validate_vote_signature` and `votable`
- Add vyper compiling instructions to README HOT 12
- Getting ready for the Vyper's beta launch HOT 4
- Suggestion: rename the repo to Casper-FFG HOT 5
- Create `voting_period` and seal off vote data HOT 1
- Deploy PURITY_CHECKER and MSG_HASHER with via compile in testing HOT 4
- Collective reward/penalties are opaque HOT 1
- Tests take too long
- Gas cost profiling, optimization, and management HOT 1
- Purity checker does not filter STATICCALL opcode HOT 7
- pytest tests error
- Intertwine Casper with lightning network
- Depracte the repo HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from casper.