No you didn’t fix it. cargo run --bin cuckatoo-miner – --tuning --edge-bits 10 --mode lean
outputs
…
Edge 8: 66 → 159 (index: 261)
Edge 9: 159 → 390 (index: 556)
…
This still violates the spec in both ways I pointed out.
To be honest, this is getting tiresome. Perhaps you should go study cuckatoo in some detail and understand how it works before you again claim to have implemented it correctly.
There are multiple codebases that already implemented a correct cycle verifier so there’s no excuse for you to keep using your own incorrect verifier and present cycles that fail on correct verifiers.
Trying is appreciated. However, reviewing code also takes time and energy. If the burden on Tromp for repetitiously reviewing the code and explaining requirements becomes higher than the benefits…
It is quite a difficult problem to solve to move grin forward, chicken and egg problem. We need experienced developers willing to long term commit to the project. To get developers to long term commit and build knowledge of the code base requires them to have very high skills already or it will require patience and time from those few experienced developers who are in very short supply. That is why I just started working on some part of the code in the hope that one day I can make some useful contributions. Start with the commitment (unpaid work to familiarize yourself), and hopefully skills and knowledge of the code base will grow over time which optionally can result in paid work in the future.
@Thomas Please only ask for reviews when you have implemented a validator that you have tested yourself to have the same output as those in the reference code. There is no hurry from our side for you to produce working code, take your time to get it right and only then ask for a review.
do you know that Tromp invented cuckatoo cycle? Please do not try to cheat with “no results found”. What is this? I like that you want to help, but think of it as a business.
Firstly, mad props to @tromp for his world changing work! Also for helping this guy learn his algorithm, your work is paramount!
Secondly, as a vibe coder myself, I feel it worth mentioning that 5 years ago, the term “vibe coding” didn’t exist, and I feel in 5 years from now, if your not vibe coding, you’re not even in the game!
I give credit to anyone working on the grin project, and prejudices against vibe coders is gonna look hilarious when looked back upon in time.
so spare me the interrogation of “are you vibe coding” and instead let’s thank the dude for his efforts, I don’t see you converting it to rust do I?
I’m not judging vibe coding, I think a lot of coding will converge to an automated form in the next 5 years. But I am judging the approach I’ve seen which made me wonder (and ask) if it was a vibe coding loop.
There are at least two things that I think need improvement here:
Review and run the code before you ask others to review it - I get that we all make mistakes, but code is runnable and this should be the minimum effort before giving it to others in review
Explain a bit more your thought process, ask questions if need be to further your understanding
Right, I should’ve written the previous reply in a nicer tone, but reading the exchange triggered me a bit.
There is no need for politeness here. This is the most obvious AI slop right from the first posting, from the made up meaningless ‘acceptance criteria’ to the em-dashes that even scammers know to remove from AI generated text these days.
I run an engineering department that uses AI tooling extensively for development. Anyone can produce a ton of AI slop that looks plausible to those who can’t look any deeper. This is not the work of an engineer, and I’d fire anyone who presented this and tried to pass it off.
It’s extremely rude and outright scummy to expect someone to spend a ton of time reviewing something you took a couple of seconds to generate with an LLM with no further thought or attempt to understand.
So, as I say, absolutely no need to be polite, this is a hostile action we’re responding to. Take this as a lesson and a warning. This is a scam attempt and something the community needs to be able to recognize and look out for as part of the process of agreeing to funding requests.
Fuck the censorship these flags are outta control!
How can you @Yeastplume accuse a guy of producing “ai slop” while at the same time bragging about using ai tooling, which is just fancy speak for ai slop.
just because his ai slop is more soupy than yours he’s a scammer? lol if you say so.
Since you say there’s no need to be polite I feel it’s worth mentioning that you abandoned this project and community! You decide to rear your head after 6 months to chop this guy down at the knees? You gonna start helping again or are you just here to criticize others attempts?
I agree @osnard the flagging is just getting silly now. Don’t worry though, most of the people in charge of this forum hold less grin than my mother. This is just a forum, they can’t touch us on-chain
There’s a big difference in you navigating the AI or the AI navigating you. It can be a powerful tool in the former and a slop in the latter.
Right now this work lacks on many fronts for a sane review. If you believe it’s on the right track then you’re more than welcome to help improve it. But people on here will expect some level of code soundness before spending substantial time on the review. Nobody is paid for reviewing nor does anyone owe the Grin community anything. It’s all free time of contributors and that time should be respected. The responses you have on here suggest that experts on the topic should invest a ton of their free time to review what is published by an AI and not checked by the user that asked the AI to code it. That’s not going to happen. Nobody has time for this because everyone (rightfully) values their free time.
And it would be great if we showed a bit of respect to the people that invested years of hard work in the project. It’s not easy work and it should not be forgotten this easily.