Comment on Rabbit R1 AI box revealed to just be an Android app
best_username_ever@sh.itjust.works 6 months agoWe’re doomed then because I would reject that in a MR for being unprofessional and full of bugs.
Comment on Rabbit R1 AI box revealed to just be an Android app
best_username_ever@sh.itjust.works 6 months agoWe’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.