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

F3/fix/hev exceeds max soc bug #143

Merged
merged 26 commits into from
Jul 11, 2024
Merged

Conversation

calbaker
Copy link
Collaborator

No description provided.

due to not having assumed e-machine has efficiency curve that is symmetrical about zero power
```warning: unused import: `super::imports::*`
 --> fastsim-core/src/air_properties.rs:1:5
  |
1 | use super::imports::*;
  |     ^^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: constant `TOL` is never used
 --> fastsim-core/src/vehicle/powertrain/fuel_converter.rs:5:7
  |
5 | const TOL: f64 = 1e-3;
  |       ^^^
  |
  = note: `#[warn(dead_code)]` on by default

warning: `fastsim-core` (lib test) generated 2 warnings (run `cargo fix --lib -p fastsim-core --tests` to apply 1 suggestion)
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.13s
     Running unittests src/lib.rs (/Users/cbaker2/Documents/github/fastsim-3/target/debug/deps/fastsim_core-3d30028b72332929)

running 1 test
test simdrive::tests::test_sim_drive_hev ... FAILED

failures:

---- simdrive::tests::test_sim_drive_hev stdout ----
thread 'simdrive::tests::test_sim_drive_hev' panicked at fastsim-core/src/utils.rs:147:24:
index out of bounds: the len is 2 but the index is 2
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    simdrive::tests::test_sim_drive_hev

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 34 filtered out; finished in 0.02s

error: test failed, to rerun pass `--lib`
```
…xceeds-max-soc-bug

# Conflicts:
#	fastsim-core/src/simdrive.rs
#	fastsim-core/src/traits.rs
@calbaker calbaker merged commit d8b6635 into fastsim-3 Jul 11, 2024
6 checks passed
@calbaker calbaker deleted the f3/fix/hev-exceeds-max-soc-bug branch July 17, 2024 16:49
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

Successfully merging this pull request may close these issues.

3 participants