Skip to content
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

Could not find a train file at path for csv with multiline values #2

Closed
marionebl opened this issue Jun 23, 2021 · 1 comment
Closed

Comments

@marionebl
Copy link

marionebl commented Jun 23, 2021

Given the following input file

content,label
a
,0
b,1

and the tangram invocation

tangram train --file input.csv --target label --output output.tangram

the following error is produced

🤔 Inferring train table columns. 0B / 23B 0% 0ms elapsed
[                                                                              ]
error: Could not find a train file at path: "input.csv"
   0: backtrace::capture::Backtrace::new
   1: tangram_core::train::Trainer::prepare
   2: tangram::train::train::{{closure}}
   3: tangram::main
   4: std::sys_common::backtrace::__rust_begin_short_backtrace
   5: _main


A ; delimited mulitline csv yields a different exception

✅ Inferring train table columns. 0ms
✅ Loading train table. 0ms
✅ Shuffling. 0ms
✅ Computing train stats. 0ms
error: panicked at 'called `Result::unwrap()` on an `Err` value: Any', cli/train.rs:158:44
   0: backtrace::capture::Backtrace::new
   1: tangram::train::train::{{closure}}
   2: std::panicking::rust_panic_with_hook
   3: std::panicking::begin_panic_handler::{{closure}}
   4: std::sys_common::backtrace::__rust_end_short_backtrace
   5: _rust_begin_unwind
   6: core::panicking::panic_fmt
   7: core::option::expect_none_failed
   8: core::ptr::drop_in_place<core::option::Option<tangram::train::ProgressThread>>
   9: tangram::train::train::{{closure}}
  10: tangram::main
  11: std::sys_common::backtrace::__rust_begin_short_backtrace
  12: _main

   0: backtrace::capture::Backtrace::new
   1: tangram::main
   2: std::sys_common::backtrace::__rust_begin_short_backtrace
   3: _main
@isabella
Copy link
Contributor

Hi @marionebl, thanks for filing this report! In version 0.3.0 tangram was providing a very poor message to CSV parsing errors. In version 0.4.0, the proper error message from the csv crate is now being bubbled up. Please re-open this issue if you continue to have a problem after upgrading. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants