-
Notifications
You must be signed in to change notification settings - Fork 90
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
This is literally garbage. you made garbage #2
Comments
@iwiwi @lerrytang @hardmaru @mkshing ^ Im assuming these are the researchers on the team at SakanaAi, since those are the only names that came up when I @ by default |
are you saying the code on the repo isn't doing what the readme says and/or doesn't meet the paper's expectations? I was interested in this as well, but when I ran it, was told my gpu wasn't up to the compute requirements. |
@thistleknot the code does exactly what the README says. Unfortunately this is just the evaluation code, to evaluate the models they created if you want. But they do say this:
So let's hope that this includes the evolutionary algorithms they used. It's a great project and there's no reason to anyone to dunk on the authors. But we CAN watch this repo and hope that they launch the research code, or someone else recreates it. If that's what you're looking for I suggest you to "Watch" this repo rather than give it a "Star" just yet, and to upvote the first comment on #3 so the authors see that there's interested people out there. In the meanwhile, there's some people trying to implement their discoveries here on github: Though an Official implementation would be very appreciated. |
I somewhat agree with what you're saying. |
Releasing a github page meant to share a new merging method, only to provide no merging code whatsoever is a really shitty move. Its not so much the fact that you didnt release your code that sucks. Its that you acted like you were going to, and put up a front like you are going to open source your methods, and then clearly only did it for popularity. Just a waste of an hour I spend researching this. Good job. 👎👎
The text was updated successfully, but these errors were encountered: