Comment on Rabbit R1 AI box revealed to just be an Android app
best_username_ever@sh.itjust.works 6 months agoThe code is bad and I would not approve this. I don’t know how you think it’s a good example for LLMs.
Comment on Rabbit R1 AI box revealed to just be an Android app
best_username_ever@sh.itjust.works 6 months agoThe code is bad and I would not approve this. I don’t know how you think it’s a good example for LLMs.
force@lemmy.world 6 months ago
The code looks like any other Python code out there.
best_username_ever@sh.itjust.works 6 months ago
We’re doomed then because I would reject that in a MR for being unprofessional and full of bugs.
FlorianSimon@sh.itjust.works 6 months ago
What bug have you spotted?
best_username_ever@sh.itjust.works 6 months ago
In one of those weird return None combination. Also I don’t get why it insist of using try catch all the time. Last but not least, it should have been one script only with sub commands using argparse, that way you could refactor most of the code.
Also weird license, overly complicated code, not handling HTTPS properly, not handling errors, a strange retry mechanism (copy pasted I guess).
It’s like a bad hack written in a hurry, or something a junior would write. Something that should never be used in production. My other gripe is that OP didn’t learn anything and wasted his time. Next time he’ll do that again and won’t improve. It’s good if he’s doing that alone, but in a company I would have to fix all this and it’s really annoying.