Refactor fn split

[lonesometraveler]

Now, this is not a simple clippy fix. I think you want to split a &str with ':' and ',' and take out &strs that represent f32. This PR should do that. Here is a set of simple tests I used. If I am missing something, please let me know.

#[test]
fn test_split() {
    let input = "foo: bar, baz, qux: -0.1, 0.2, -0.3";
    let expected_output = vec!["-0.1", "0.2", "-0.3"];
    let actual_output = split(input);
    assert_eq!(expected_output, actual_output);
}

#[test]
fn test_split_2() {
    let input = "foo:bar,baz,qux:-0.1,0.2,-0.3";
    let expected_output = vec!["-0.1", "0.2", "-0.3"];
    let actual_output = split(input);
    assert_eq!(expected_output, actual_output);
}

I think we should parse in this function and return Vec<f32> instead of Vec<&str>. But that requires modification in main. Let's make sure we are good with this change first.

[hacknus]

Yes, perfect! This looks much cleaner and seems to work. I agree about returning f32 and adapt main().

[lonesometraveler]

Cool. I will send more PRs tomorrow.