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

panic at index out of bounds #9

Closed
matthiasbeyer opened this issue Jul 29, 2024 · 11 comments
Closed

panic at index out of bounds #9

matthiasbeyer opened this issue Jul 29, 2024 · 11 comments
Labels
need more info Lacks enough info to make progress

Comments

@matthiasbeyer
Copy link

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
Copy link
Owner

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

@matthiasbeyer
Copy link
Author

matthiasbeyer commented Jul 29, 2024

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
Copy link
Owner

Probably the underlying problem is the same as #2.

@juliolinarez
Copy link

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
Copy link

❯ 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
Copy link
Owner

lusingander commented Jul 29, 2024

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
Copy link
Contributor

alerque commented Jul 29, 2024

Same problem here, but only on some repos.

@lusingander lusingander added the need more info Lacks enough info to make progress label Jul 29, 2024
@AgustinBerardi
Copy link

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:
Screenshot 2024-07-29 at 11 01 06 PM

@lusingander
Copy link
Owner

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
Copy link
Owner

lusingander commented Aug 1, 2024

I'm not entirely sure what your specific problem is, but #5 has been merged so that might solve your problem.

The fix for #5 was not properly reflected in the release 😢
#20

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

@lusingander
Copy link
Owner

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need more info Lacks enough info to make progress
Projects
None yet
Development

No branches or pull requests

5 participants