panic at index out of bounds

matthiasbeyer commented 3 months ago

thread 'main' panicked at src/graph/calc.rs:228:5:                                                                                                                                                 
index out of bounds: the len is 5 but the index is 5                                                                                                                                               
stack backtrace:                                                                                                                                                                                   
   0: rust_begin_unwind                                                                                                                                                                            
   1: core::panicking::panic_fmt                                                                                                                                                                   
   2: core::panicking::panic_bounds_check                                                                                                                                                          
   3: serie::graph::calc::calc_graph                                                                                                                                                               
   4: serie::run                                                                                                                                                                                   
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

lusingander commented 3 months ago

What repository are you running? I don't know the details so there's nothing I can do.

matthiasbeyer commented 3 months ago

I am running 0.1.0

Edit: Ah, you mean on what repo I run this? Well, that's proprietary, so I cannot share... Still, this is simple as this is an index into a slice, which panics because of out of bounds... simple to fix I guess, if you know the codebase.

lusingander commented 3 months ago

Probably the underlying problem is the same as #2.

juliolinarez commented 3 months ago

Same error. (serie v0.1.0) (installed with brew). Private repository

thread 'main' panicked at src/graph/calc.rs:228:5:
index out of bounds: the len is 47 but the index is 47
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

juliolinarez commented 3 months ago

❯ RUST_BACKTRACE=1 serie
thread 'main' panicked at src/graph/calc.rs:228:5:
index out of bounds: the len is 47 but the index is 47
stack backtrace:
   0: _rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::panicking::panic_bounds_check
   3: serie::graph::calc::calc_graph
   4: serie::run
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
❯ RUST_BACKTRACE=full serie
thread 'main' panicked at src/graph/calc.rs:228:5:
index out of bounds: the len is 47 but the index is 47
stack backtrace:
   0:        0x10313744c - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h1f3776e0b5c7517d
   1:        0x103155128 - core::fmt::write::heedef092c8c0962e
   2:        0x10313471c - std::io::Write::write_fmt::h7178e8e2ea928914
   3:        0x1031372a4 - std::sys_common::backtrace::print::h417292deb95532ed
   4:        0x1031386a8 - std::panicking::default_hook::{{closure}}::h0cb68f1228c4613a
   5:        0x10313839c - std::panicking::default_hook::h24535936bc1f51de
   6:        0x103138f60 - std::panicking::rust_panic_with_hook::h5db4d2345b297bed
   7:        0x103138990 - std::panicking::begin_panic_handler::{{closure}}::h3fd558f09a0d5492
   8:        0x1031378d4 - std::sys_common::backtrace::__rust_end_short_backtrace::hfc76eebe1ce501b2
   9:        0x103138700 - _rust_begin_unwind
  10:        0x10317bf84 - core::panicking::panic_fmt::hc2b459a5bd3dce66
  11:        0x10317c0cc - core::panicking::panic_bounds_check::hd61643b086b4289f
  12:        0x102ed8e34 - serie::graph::calc::calc_graph::h9e19fe0ce74cbc11
  13:        0x102f1069c - serie::run::h946d0ce14d7749c3
  14:        0x102dc55f8 - std::sys_common::backtrace::__rust_begin_short_backtrace::hc73c8b47148a5084
  15:        0x102dc5618 - std::rt::lang_start::{{closure}}::hdaafe243cf43680d
  16:        0x10312e318 - std::rt::lang_start_internal::hecc68fef83c8f44d
  17:        0x102dc5804 - _main

lusingander commented 3 months ago

I'm not entirely sure what your specific problem is, but #5 has been merged so that might solve your problem. ~~It hasn't been released yet so please build it if you need it.~~ released: https://github.com/lusingander/serie/releases/tag/v0.1.1

alerque commented 3 months ago

Same problem here, but only on some repos.

AgustinBerardi commented 3 months ago

I had the same issue before the last release (version 0.1.0). With the new update (v0.1.1), here is the new error:

lusingander commented 3 months ago

If you just post a screenshot, it doesn't tell us anything.

Please clarify the terminal and version you are using, the protocol you are specifying, etc.
Please provide a repository for reproducing the issue.
- Does it work with a simple repository?

lusingander commented 3 months ago

I'm not entirely sure what your specific problem is, but https://github.com/lusingander/serie/pull/5 has been merged so that might solve your problem.

The fix for #5 was not properly reflected in the release 😢 https://github.com/lusingander/serie/pull/20

released: https://github.com/lusingander/serie/releases/tag/v0.1.2

lusingander commented 3 months ago

Since there is no additional information, we will close this issue for now. If necessary, please reopen or create a new issue with information to reproduce it.

lusingander / serie

panic at index out of bounds #9