7 kyu

Lojban Numbers

448 of 565alexlostorto
Description
Loading description...
Strings
Algorithms
Parsing
View
AllIssues1QuestionsSuggestions2Show Resolved
  • Please sign in or sign up to leave a comment.
  • Unnamed Avatar

    Rust:

    error[E0425]: cannot find function `covert_lojban` in this scope
  --> src/lib.rs:11:20
   |
1  | fn convert_lojban(input: &str) -> u64 {
   | ------------------------------------- similarly named function `convert_lojban` defined here
...

    and the sample tests don't have #[test].

    • alexlostorto Avatar

      Just wondering, someone else wrote that translation (not me) so it will be pretty difficult for me to fix it. How should I go about resolving this?

      Is it possible to remove the translation entirely?

    • alexlostorto Avatar

      Just added #[test] at the top of the sample tests file, everything seems to work now.

      Please open another issue if this persists!

      Issue marked resolved by alexlostorto 15 months ago
    • Unnamed Avatar

      The name in the tests still is covert_lojban.

    • alexlostorto Avatar

      LMAO Just spotted that haha! I believe I have fixed it now in case you want to check it again :)

      BTW, is it not possible for you to fix the tests yourself? And if it is, do you make the kata author do it so they gain experience or just so you have less work to do?

    • rowcased Avatar

      IMHO the work they're doing to inform you in a detailed way about what your kata needs to be fixed is a fair contrubution all on its own

    • Unnamed Avatar

      Becoming a kata contributor has some negative side effects affecting progress in individual languages. Some bugs have been fixed, I don't remember how it works now, but I think contributors don't have a confirmation pop-up before opening solutions in any language and there are many languageless links on the site, so I regularly open solution pages in the wrong languages.

    • alexlostorto Avatar

      Sounds pretty confusing, thanks for both your responses 👍

  • dfhwze Avatar

    Why not store the translation table as rows instead of this grid?

    0 no
1 pa 
2 re 
3 ci 
4 vo 
5 mu 
6 xa
7 ze
8 bi 
9 so
    • alexlostorto Avatar

      It's more compact, and also it's just how it was displayed in the real BIO question - so I've been trying to keep everything as close as possible to make it more realistic.