[INFO] cloning repository https://github.com/abhatia1205/neural-ink-interview-2
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/abhatia1205/neural-ink-interview-2" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fabhatia1205%2Fneural-ink-interview-2", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fabhatia1205%2Fneural-ink-interview-2'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 45418261dfa881f1a711a3b1148bac2edcbc47d2
[INFO] testing abhatia1205/neural-ink-interview-2/45418261dfa881f1a711a3b1148bac2edcbc47d2 against master#d98a5da813da67eb189387b8ccfb73cf481275d8+rustflags=-Copt-level=3 for pr-138759
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fabhatia1205%2Fneural-ink-interview-2" "/workspace/builds/worker-7-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/abhatia1205/neural-ink-interview-2
[INFO] finished tweaking git repo https://github.com/abhatia1205/neural-ink-interview-2
[INFO] tweaked toml for git repo https://github.com/abhatia1205/neural-ink-interview-2 written to /workspace/builds/worker-7-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/abhatia1205/neural-ink-interview-2 on toolchain d98a5da813da67eb189387b8ccfb73cf481275d8
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/abhatia1205/neural-ink-interview-2 already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded roots v0.0.8
[INFO] [stderr]   Downloaded libc v0.2.166
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] 34609293ff176af4237592119d428457de30d4a728013b17b81fb0fb77969e23
[INFO] running `Command { std: "docker" "start" "-a" "34609293ff176af4237592119d428457de30d4a728013b17b81fb0fb77969e23", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "34609293ff176af4237592119d428457de30d4a728013b17b81fb0fb77969e23", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "34609293ff176af4237592119d428457de30d4a728013b17b81fb0fb77969e23", kill_on_drop: false }`
[INFO] [stdout] 34609293ff176af4237592119d428457de30d4a728013b17b81fb0fb77969e23
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Copt-level=3" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "build" "--frozen" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] d22f23c90eba3fa6f7204f4a45ec40675b5ac8f6f6a78569cf9880edf8bb9dbd
[INFO] running `Command { std: "docker" "start" "-a" "d22f23c90eba3fa6f7204f4a45ec40675b5ac8f6f6a78569cf9880edf8bb9dbd", kill_on_drop: false }`
[INFO] [stderr]    Compiling proc-macro2 v1.0.92
[INFO] [stderr]    Compiling libc v0.2.166
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]    Compiling lock_api v0.4.12
[INFO] [stderr]    Compiling bytemuck v1.20.0
[INFO] [stderr]    Compiling matrixmultiply v0.3.9
[INFO] [stderr]    Compiling smallvec v1.13.2
[INFO] [stderr]    Compiling rawpointer v0.2.1
[INFO] [stderr]    Compiling bytes v1.9.0
[INFO] [stderr]    Compiling safe_arch v0.7.2
[INFO] [stderr]    Compiling roots v0.0.8
[INFO] [stderr]    Compiling wide v0.7.30
[INFO] [stderr]    Compiling quote v1.0.37
[INFO] [stderr]    Compiling syn v2.0.89
[INFO] [stderr]    Compiling getrandom v0.2.15
[INFO] [stderr]    Compiling parking_lot_core v0.9.10
[INFO] [stderr]    Compiling rand_core v0.6.4
[INFO] [stderr]    Compiling approx v0.5.1
[INFO] [stderr]    Compiling num-integer v0.1.46
[INFO] [stderr]    Compiling num-complex v0.4.6
[INFO] [stderr]    Compiling parking_lot v0.12.3
[INFO] [stderr]    Compiling mio v1.0.2
[INFO] [stderr]    Compiling num-rational v0.4.2
[INFO] [stderr]    Compiling simba v0.9.0
[INFO] [stderr]    Compiling socket2 v0.5.8
[INFO] [stderr]    Compiling signal-hook-registry v1.4.2
[INFO] [stderr]    Compiling zerocopy-derive v0.7.35
[INFO] [stderr]    Compiling nalgebra-macros v0.2.2
[INFO] [stderr]    Compiling tokio-macros v2.4.0
[INFO] [stderr]    Compiling nalgebra v0.33.2
[INFO] [stderr]    Compiling tokio v1.41.1
[INFO] [stderr]    Compiling zerocopy v0.7.35
[INFO] [stderr]    Compiling ppv-lite86 v0.2.20
[INFO] [stderr]    Compiling rand_chacha v0.3.1
[INFO] [stderr]    Compiling rand v0.8.5
[INFO] [stderr]    Compiling neuralink_final v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/robot.rs:244:23
[INFO] [stdout]     |
[INFO] [stdout] 244 |                     if(z != 0){
[INFO] [stdout]     |                       ^      ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 244 -                     if(z != 0){
[INFO] [stdout] 244 +                     if z != 0 {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stdout]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stdout]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/robot.rs:220:17
[INFO] [stdout]     |
[INFO] [stdout] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stdout]     |                 ----^^^^^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:22:5
[INFO] [stdout]    |
[INFO] [stdout] 22 |     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]    = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 22 -     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stdout] 22 +     fn get_surface_distance(&self) -> impl std::future::Future<Output = Result<u64, OCTError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:72:5
[INFO] [stdout]    |
[INFO] [stdout] 72 |     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 72 -     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stdout] 72 +     fn get_robot_state(&self) -> impl std::future::Future<Output = Result<RobotState, RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:74:5
[INFO] [stdout]    |
[INFO] [stdout] 74 |     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 74 -     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stdout] 74 +     fn command_move(&self, command: &Move) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:75:5
[INFO] [stdout]    |
[INFO] [stdout] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 75 -     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout] 75 +     fn command_grasp(&self) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stdout]    |
[INFO] [stdout] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/robot.rs:244:23
[INFO] [stdout]     |
[INFO] [stdout] 244 |                     if(z != 0){
[INFO] [stdout]     |                       ^      ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 244 -                     if(z != 0){
[INFO] [stdout] 244 +                     if z != 0 {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stdout]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stdout]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `predictor::taylor_approx::TaylorQuadraticApproximator`
[INFO] [stdout]   --> src/main.rs:12:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | use predictor::taylor_approx::TaylorQuadraticApproximator;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `predictor::oracle_approx::OraclePredictor`
[INFO] [stdout]   --> src/main.rs:14:5
[INFO] [stdout]    |
[INFO] [stdout] 14 | use predictor::oracle_approx::OraclePredictor;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/robot.rs:220:17
[INFO] [stdout]     |
[INFO] [stdout] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stdout]     |                 ----^^^^^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `msg` is never read
[INFO] [stdout]  --> src/interface.rs:7:26
[INFO] [stdout]   |
[INFO] [stdout] 7 |     CommunicationError { msg: String },
[INFO] [stdout]   |     ------------------   ^^^
[INFO] [stdout]   |     |
[INFO] [stdout]   |     field in this variant
[INFO] [stdout]   |
[INFO] [stdout]   = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout]   = note: `#[warn(dead_code)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `AcquisitionError`, `TimeoutError`, and `PredictionError` are never constructed
[INFO] [stdout]   --> src/interface.rs:5:5
[INFO] [stdout]    |
[INFO] [stdout] 3  | pub enum OCTError {
[INFO] [stdout]    |          -------- variants in this enum
[INFO] [stdout] 4  |     // Failed to acquire data from the OCT laser
[INFO] [stdout] 5  |     AcquisitionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 9  |     TimeoutError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout] 10 |
[INFO] [stdout] 11 |     PredictionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `msg` is never read
[INFO] [stdout]   --> src/interface.rs:56:17
[INFO] [stdout]    |
[INFO] [stdout] 56 |     MoveError { msg: String },
[INFO] [stdout]    |     ---------   ^^^
[INFO] [stdout]    |     |
[INFO] [stdout]    |     field in this variant
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `ConnectionError` and `PositionError` are never constructed
[INFO] [stdout]   --> src/interface.rs:58:5
[INFO] [stdout]    |
[INFO] [stdout] 54 | pub enum RobotError {
[INFO] [stdout]    |          ---------- variants in this enum
[INFO] [stdout] ...
[INFO] [stdout] 58 |     ConnectionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 61 |     PositionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `command_grasp` is never used
[INFO] [stdout]   --> src/interface.rs:75:14
[INFO] [stdout]    |
[INFO] [stdout] 71 | pub trait Robot {
[INFO] [stdout]    |           ----- method in this trait
[INFO] [stdout] ...
[INFO] [stdout] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout]    |              ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `state_errors` is never read
[INFO] [stdout]   --> src/robot.rs:15:9
[INFO] [stdout]    |
[INFO] [stdout] 13 | pub struct RobotArm {
[INFO] [stdout]    |            -------- field in this struct
[INFO] [stdout] 14 |     pub distance_errors: bool,
[INFO] [stdout] 15 |     pub state_errors: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `train` is never used
[INFO] [stdout]   --> src/predictor/mod.rs:10:8
[INFO] [stdout]    |
[INFO] [stdout] 8  | pub trait BrainPredictor {
[INFO] [stdout]    |           -------------- method in this trait
[INFO] [stdout] 9  |     fn predict(&self, distances: &Vec<Result<u64, OCTError>>, times: &Vec<Instant>, print_coefs: bool) -> Option<impl Fn(f64) -> f64>;
[INFO] [stdout] 10 |     fn train(&self) -> bool{
[INFO] [stdout]    |        ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MIN_SIZE` is never used
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:5:7
[INFO] [stdout]   |
[INFO] [stdout] 5 | const MIN_SIZE: usize =3;
[INFO] [stdout]   |       ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:6:7
[INFO] [stdout]   |
[INFO] [stdout] 6 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `OraclePredictor` is never constructed
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub struct OraclePredictor{
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated functions `new` and `passes_predict_assumptions` are never used
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:13:12
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl OraclePredictor{
[INFO] [stdout]    | -------------------- associated functions in this implementation
[INFO] [stdout] 13 |     pub fn new() -> OraclePredictor{
[INFO] [stdout]    |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 19 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(Vec<u64>, Vec<Instan...
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:4:7
[INFO] [stdout]   |
[INFO] [stdout] 4 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_STD_MS` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:5:7
[INFO] [stdout]   |
[INFO] [stdout] 5 | const MAX_LATENCY_STD_MS: u64 = 3;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `TAYLOR_POLY_ORDER` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:6:7
[INFO] [stdout]   |
[INFO] [stdout] 6 | const TAYLOR_POLY_ORDER: u64 = 2; 
[INFO] [stdout]   |       ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `TaylorQuadraticApproximator` is never constructed
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub struct TaylorQuadraticApproximator;
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated function `passes_predict_assumptions` is never used
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:32:8
[INFO] [stdout]    |
[INFO] [stdout] 10 | impl TaylorQuadraticApproximator{
[INFO] [stdout]    | -------------------------------- associated function in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 32 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(f64, f64, Vec<u64>, ...
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stdout]    |
[INFO] [stdout] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 2m 21s
[INFO] running `Command { std: "docker" "inspect" "d22f23c90eba3fa6f7204f4a45ec40675b5ac8f6f6a78569cf9880edf8bb9dbd", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "d22f23c90eba3fa6f7204f4a45ec40675b5ac8f6f6a78569cf9880edf8bb9dbd", kill_on_drop: false }`
[INFO] [stdout] d22f23c90eba3fa6f7204f4a45ec40675b5ac8f6f6a78569cf9880edf8bb9dbd
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Copt-level=3" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 878008ceb3b1ee187551ba21ec444b4c8fad9cf4d78ad4cd8a695386a2b3b00e
[INFO] running `Command { std: "docker" "start" "-a" "878008ceb3b1ee187551ba21ec444b4c8fad9cf4d78ad4cd8a695386a2b3b00e", kill_on_drop: false }`
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/robot.rs:244:23
[INFO] [stdout]     |
[INFO] [stdout] 244 |                     if(z != 0){
[INFO] [stdout]     |                       ^      ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 244 -                     if(z != 0){
[INFO] [stdout] 244 +                     if z != 0 {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]    Compiling neuralink_final v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stdout]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stdout]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/robot.rs:220:17
[INFO] [stdout]     |
[INFO] [stdout] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stdout]     |                 ----^^^^^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:22:5
[INFO] [stdout]    |
[INFO] [stdout] 22 |     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]    = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 22 -     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stdout] 22 +     fn get_surface_distance(&self) -> impl std::future::Future<Output = Result<u64, OCTError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:72:5
[INFO] [stdout]    |
[INFO] [stdout] 72 |     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 72 -     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stdout] 72 +     fn get_robot_state(&self) -> impl std::future::Future<Output = Result<RobotState, RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:74:5
[INFO] [stdout]    |
[INFO] [stdout] 74 |     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 74 -     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stdout] 74 +     fn command_move(&self, command: &Move) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:75:5
[INFO] [stdout]    |
[INFO] [stdout] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 75 -     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout] 75 +     fn command_grasp(&self) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stdout]    |
[INFO] [stdout] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/robot.rs:244:23
[INFO] [stdout]     |
[INFO] [stdout] 244 |                     if(z != 0){
[INFO] [stdout]     |                       ^      ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 244 -                     if(z != 0){
[INFO] [stdout] 244 +                     if z != 0 {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stdout]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stdout]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `predictor::taylor_approx::TaylorQuadraticApproximator`
[INFO] [stdout]   --> src/main.rs:12:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | use predictor::taylor_approx::TaylorQuadraticApproximator;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `predictor::oracle_approx::OraclePredictor`
[INFO] [stdout]   --> src/main.rs:14:5
[INFO] [stdout]    |
[INFO] [stdout] 14 | use predictor::oracle_approx::OraclePredictor;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/robot.rs:220:17
[INFO] [stdout]     |
[INFO] [stdout] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stdout]     |                 ----^^^^^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `msg` is never read
[INFO] [stdout]  --> src/interface.rs:7:26
[INFO] [stdout]   |
[INFO] [stdout] 7 |     CommunicationError { msg: String },
[INFO] [stdout]   |     ------------------   ^^^
[INFO] [stdout]   |     |
[INFO] [stdout]   |     field in this variant
[INFO] [stdout]   |
[INFO] [stdout]   = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout]   = note: `#[warn(dead_code)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `AcquisitionError`, `TimeoutError`, and `PredictionError` are never constructed
[INFO] [stdout]   --> src/interface.rs:5:5
[INFO] [stdout]    |
[INFO] [stdout] 3  | pub enum OCTError {
[INFO] [stdout]    |          -------- variants in this enum
[INFO] [stdout] 4  |     // Failed to acquire data from the OCT laser
[INFO] [stdout] 5  |     AcquisitionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 9  |     TimeoutError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout] 10 |
[INFO] [stdout] 11 |     PredictionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `msg` is never read
[INFO] [stdout]   --> src/interface.rs:56:17
[INFO] [stdout]    |
[INFO] [stdout] 56 |     MoveError { msg: String },
[INFO] [stdout]    |     ---------   ^^^
[INFO] [stdout]    |     |
[INFO] [stdout]    |     field in this variant
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `ConnectionError` and `PositionError` are never constructed
[INFO] [stdout]   --> src/interface.rs:58:5
[INFO] [stdout]    |
[INFO] [stdout] 54 | pub enum RobotError {
[INFO] [stdout]    |          ---------- variants in this enum
[INFO] [stdout] ...
[INFO] [stdout] 58 |     ConnectionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 61 |     PositionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `command_grasp` is never used
[INFO] [stdout]   --> src/interface.rs:75:14
[INFO] [stdout]    |
[INFO] [stdout] 71 | pub trait Robot {
[INFO] [stdout]    |           ----- method in this trait
[INFO] [stdout] ...
[INFO] [stdout] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout]    |              ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `state_errors` is never read
[INFO] [stdout]   --> src/robot.rs:15:9
[INFO] [stdout]    |
[INFO] [stdout] 13 | pub struct RobotArm {
[INFO] [stdout]    |            -------- field in this struct
[INFO] [stdout] 14 |     pub distance_errors: bool,
[INFO] [stdout] 15 |     pub state_errors: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `train` is never used
[INFO] [stdout]   --> src/predictor/mod.rs:10:8
[INFO] [stdout]    |
[INFO] [stdout] 8  | pub trait BrainPredictor {
[INFO] [stdout]    |           -------------- method in this trait
[INFO] [stdout] 9  |     fn predict(&self, distances: &Vec<Result<u64, OCTError>>, times: &Vec<Instant>, print_coefs: bool) -> Option<impl Fn(f64) -> f64>;
[INFO] [stdout] 10 |     fn train(&self) -> bool{
[INFO] [stdout]    |        ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MIN_SIZE` is never used
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:5:7
[INFO] [stdout]   |
[INFO] [stdout] 5 | const MIN_SIZE: usize =3;
[INFO] [stdout]   |       ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:6:7
[INFO] [stdout]   |
[INFO] [stdout] 6 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `OraclePredictor` is never constructed
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub struct OraclePredictor{
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated functions `new` and `passes_predict_assumptions` are never used
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:13:12
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl OraclePredictor{
[INFO] [stdout]    | -------------------- associated functions in this implementation
[INFO] [stdout] 13 |     pub fn new() -> OraclePredictor{
[INFO] [stdout]    |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 19 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(Vec<u64>, Vec<Instan...
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:4:7
[INFO] [stdout]   |
[INFO] [stdout] 4 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_STD_MS` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:5:7
[INFO] [stdout]   |
[INFO] [stdout] 5 | const MAX_LATENCY_STD_MS: u64 = 3;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `TAYLOR_POLY_ORDER` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:6:7
[INFO] [stdout]   |
[INFO] [stdout] 6 | const TAYLOR_POLY_ORDER: u64 = 2; 
[INFO] [stdout]   |       ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `TaylorQuadraticApproximator` is never constructed
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub struct TaylorQuadraticApproximator;
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated function `passes_predict_assumptions` is never used
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:32:8
[INFO] [stdout]    |
[INFO] [stdout] 10 | impl TaylorQuadraticApproximator{
[INFO] [stdout]    | -------------------------------- associated function in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 32 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(f64, f64, Vec<u64>, ...
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stdout]    |
[INFO] [stdout] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/robot.rs:244:23
[INFO] [stdout]     |
[INFO] [stdout] 244 |                     if(z != 0){
[INFO] [stdout]     |                       ^      ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 244 -                     if(z != 0){
[INFO] [stdout] 244 +                     if z != 0 {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stdout]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stdout]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around `if` condition
[INFO] [stdout]    --> src/robot.rs:244:23
[INFO] [stdout]     |
[INFO] [stdout] 244 |                     if(z != 0){
[INFO] [stdout]     |                       ^      ^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]     |
[INFO] [stdout] 244 -                     if(z != 0){
[INFO] [stdout] 244 +                     if z != 0 {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stdout]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stdout]   |
[INFO] [stdout] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stdout]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `predictor::taylor_approx::TaylorQuadraticApproximator`
[INFO] [stdout]   --> src/main.rs:12:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | use predictor::taylor_approx::TaylorQuadraticApproximator;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `predictor::oracle_approx::OraclePredictor`
[INFO] [stdout]   --> src/main.rs:14:5
[INFO] [stdout]    |
[INFO] [stdout] 14 | use predictor::oracle_approx::OraclePredictor;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/robot.rs:220:17
[INFO] [stdout]     |
[INFO] [stdout] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stdout]     |                 ----^^^^^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/robot.rs:220:17
[INFO] [stdout]     |
[INFO] [stdout] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stdout]     |                 ----^^^^^^^^^^
[INFO] [stdout]     |                 |
[INFO] [stdout]     |                 help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:22:5
[INFO] [stdout]    |
[INFO] [stdout] 22 |     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]    = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 22 -     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stdout] 22 +     fn get_surface_distance(&self) -> impl std::future::Future<Output = Result<u64, OCTError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:72:5
[INFO] [stdout]    |
[INFO] [stdout] 72 |     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 72 -     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stdout] 72 +     fn get_robot_state(&self) -> impl std::future::Future<Output = Result<RobotState, RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:74:5
[INFO] [stdout]    |
[INFO] [stdout] 74 |     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 74 -     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stdout] 74 +     fn command_move(&self, command: &Move) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]   --> src/interface.rs:75:5
[INFO] [stdout]    |
[INFO] [stdout] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]    |
[INFO] [stdout] 75 -     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout] 75 +     fn command_grasp(&self) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stdout]    |
[INFO] [stdout] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `msg` is never read
[INFO] [stdout]  --> src/interface.rs:7:26
[INFO] [stdout]   |
[INFO] [stdout] 7 |     CommunicationError { msg: String },
[INFO] [stdout]   |     ------------------   ^^^
[INFO] [stdout]   |     |
[INFO] [stdout]   |     field in this variant
[INFO] [stdout]   |
[INFO] [stdout]   = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout]   = note: `#[warn(dead_code)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `AcquisitionError`, `TimeoutError`, and `PredictionError` are never constructed
[INFO] [stdout]   --> src/interface.rs:5:5
[INFO] [stdout]    |
[INFO] [stdout] 3  | pub enum OCTError {
[INFO] [stdout]    |          -------- variants in this enum
[INFO] [stdout] 4  |     // Failed to acquire data from the OCT laser
[INFO] [stdout] 5  |     AcquisitionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 9  |     TimeoutError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout] 10 |
[INFO] [stdout] 11 |     PredictionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `msg` is never read
[INFO] [stdout]   --> src/interface.rs:56:17
[INFO] [stdout]    |
[INFO] [stdout] 56 |     MoveError { msg: String },
[INFO] [stdout]    |     ---------   ^^^
[INFO] [stdout]    |     |
[INFO] [stdout]    |     field in this variant
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `ConnectionError` and `PositionError` are never constructed
[INFO] [stdout]   --> src/interface.rs:58:5
[INFO] [stdout]    |
[INFO] [stdout] 54 | pub enum RobotError {
[INFO] [stdout]    |          ---------- variants in this enum
[INFO] [stdout] ...
[INFO] [stdout] 58 |     ConnectionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 61 |     PositionError { msg: String },
[INFO] [stdout]    |     ^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `command_grasp` is never used
[INFO] [stdout]   --> src/interface.rs:75:14
[INFO] [stdout]    |
[INFO] [stdout] 71 | pub trait Robot {
[INFO] [stdout]    |           ----- method in this trait
[INFO] [stdout] ...
[INFO] [stdout] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stdout]    |              ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `state_errors` is never read
[INFO] [stdout]   --> src/robot.rs:15:9
[INFO] [stdout]    |
[INFO] [stdout] 13 | pub struct RobotArm {
[INFO] [stdout]    |            -------- field in this struct
[INFO] [stdout] 14 |     pub distance_errors: bool,
[INFO] [stdout] 15 |     pub state_errors: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `train` is never used
[INFO] [stdout]   --> src/predictor/mod.rs:10:8
[INFO] [stdout]    |
[INFO] [stdout] 8  | pub trait BrainPredictor {
[INFO] [stdout]    |           -------------- method in this trait
[INFO] [stdout] 9  |     fn predict(&self, distances: &Vec<Result<u64, OCTError>>, times: &Vec<Instant>, print_coefs: bool) -> Option<impl Fn(f64) -> f64>;
[INFO] [stdout] 10 |     fn train(&self) -> bool{
[INFO] [stdout]    |        ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MIN_SIZE` is never used
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:5:7
[INFO] [stdout]   |
[INFO] [stdout] 5 | const MIN_SIZE: usize =3;
[INFO] [stdout]   |       ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:6:7
[INFO] [stdout]   |
[INFO] [stdout] 6 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `OraclePredictor` is never constructed
[INFO] [stdout]  --> src/predictor/oracle_approx.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub struct OraclePredictor{
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated functions `new` and `passes_predict_assumptions` are never used
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:13:12
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl OraclePredictor{
[INFO] [stdout]    | -------------------- associated functions in this implementation
[INFO] [stdout] 13 |     pub fn new() -> OraclePredictor{
[INFO] [stdout]    |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 19 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(Vec<u64>, Vec<Instan...
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:4:7
[INFO] [stdout]   |
[INFO] [stdout] 4 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `MAX_LATENCY_STD_MS` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:5:7
[INFO] [stdout]   |
[INFO] [stdout] 5 | const MAX_LATENCY_STD_MS: u64 = 3;
[INFO] [stdout]   |       ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `TAYLOR_POLY_ORDER` is never used
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:6:7
[INFO] [stdout]   |
[INFO] [stdout] 6 | const TAYLOR_POLY_ORDER: u64 = 2; 
[INFO] [stdout]   |       ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `TaylorQuadraticApproximator` is never constructed
[INFO] [stdout]  --> src/predictor/taylor_approx.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 | pub struct TaylorQuadraticApproximator;
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated function `passes_predict_assumptions` is never used
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:32:8
[INFO] [stdout]    |
[INFO] [stdout] 10 | impl TaylorQuadraticApproximator{
[INFO] [stdout]    | -------------------------------- associated function in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 32 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(f64, f64, Vec<u64>, ...
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stdout]    |
[INFO] [stdout] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stdout] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `data_len` should have an upper case name
[INFO] [stdout]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stdout]    |
[INFO] [stdout] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout] help: convert the identifier to upper case
[INFO] [stdout]    |
[INFO] [stdout] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `test` profile [unoptimized + debuginfo] target(s) in 44.25s
[INFO] running `Command { std: "docker" "inspect" "878008ceb3b1ee187551ba21ec444b4c8fad9cf4d78ad4cd8a695386a2b3b00e", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "878008ceb3b1ee187551ba21ec444b4c8fad9cf4d78ad4cd8a695386a2b3b00e", kill_on_drop: false }`
[INFO] [stdout] 878008ceb3b1ee187551ba21ec444b4c8fad9cf4d78ad4cd8a695386a2b3b00e
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Copt-level=3" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "test" "--frozen", kill_on_drop: false }`
[INFO] [stdout] 37166713a04c70ff33a1eb12a80fc0babee90e1d6e636e28a8428799ac80f934
[INFO] running `Command { std: "docker" "start" "-a" "37166713a04c70ff33a1eb12a80fc0babee90e1d6e636e28a8428799ac80f934", kill_on_drop: false }`
[INFO] [stderr] warning: unnecessary parentheses around `if` condition
[INFO] [stderr]    --> src/robot.rs:244:23
[INFO] [stderr]     |
[INFO] [stderr] 244 |                     if(z != 0){
[INFO] [stderr]     |                       ^      ^
[INFO] [stderr]     |
[INFO] [stderr]     = note: `#[warn(unused_parens)]` on by default
[INFO] [stderr] help: remove these parentheses
[INFO] [stderr]     |
[INFO] [stderr] 244 -                     if(z != 0){
[INFO] [stderr] 244 +                     if z != 0 {
[INFO] [stderr]     |
[INFO] [stderr] 
[INFO] [stderr] warning: unused imports: `char::MAX` and `f32::MIN`
[INFO] [stderr]  --> src/predictor/quadratic_regression.rs:1:11
[INFO] [stderr]   |
[INFO] [stderr] 1 | use std::{char::MAX, f32::MIN};
[INFO] [stderr]   |           ^^^^^^^^^  ^^^^^^^^
[INFO] [stderr]   |
[INFO] [stderr]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stderr] 
[INFO] [stderr] warning: variable does not need to be mutable
[INFO] [stderr]    --> src/robot.rs:220:17
[INFO] [stderr]     |
[INFO] [stderr] 220 |             let mut will_error = guard.move_errors && rng.gen_bool(PROBABILITY_OF_ERROR);
[INFO] [stderr]     |                 ----^^^^^^^^^^
[INFO] [stderr]     |                 |
[INFO] [stderr]     |                 help: remove this `mut`
[INFO] [stderr]     |
[INFO] [stderr]     = note: `#[warn(unused_mut)]` on by default
[INFO] [stderr] 
[INFO] [stderr] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stderr]   --> src/interface.rs:22:5
[INFO] [stderr]    |
[INFO] [stderr] 22 |     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stderr]    |     ^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stderr]    = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stderr] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stderr]    |
[INFO] [stderr] 22 -     async fn get_surface_distance(&self) -> Result<u64, OCTError>;
[INFO] [stderr] 22 +     fn get_surface_distance(&self) -> impl std::future::Future<Output = Result<u64, OCTError>> + Send;
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stderr]   --> src/interface.rs:72:5
[INFO] [stderr]    |
[INFO] [stderr] 72 |     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stderr]    |     ^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stderr] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stderr]    |
[INFO] [stderr] 72 -     async fn get_robot_state(&self) -> Result<RobotState, RobotError>;
[INFO] [stderr] 72 +     fn get_robot_state(&self) -> impl std::future::Future<Output = Result<RobotState, RobotError>> + Send;
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stderr]   --> src/interface.rs:74:5
[INFO] [stderr]    |
[INFO] [stderr] 74 |     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stderr]    |     ^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stderr] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stderr]    |
[INFO] [stderr] 74 -     async fn command_move(&self, command: &Move) -> Result<(), RobotError>;
[INFO] [stderr] 74 +     fn command_move(&self, command: &Move) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stderr]   --> src/interface.rs:75:5
[INFO] [stderr]    |
[INFO] [stderr] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stderr]    |     ^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stderr] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stderr]    |
[INFO] [stderr] 75 -     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stderr] 75 +     fn command_grasp(&self) -> impl std::future::Future<Output = Result<(), RobotError>> + Send;
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: constant `data_len` should have an upper case name
[INFO] [stderr]   --> src/predictor/oracle_approx.rs:20:15
[INFO] [stderr]    |
[INFO] [stderr] 20 |         const data_len: usize = MIN_SIZE+1;
[INFO] [stderr]    |               ^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `#[warn(non_upper_case_globals)]` on by default
[INFO] [stderr] help: convert the identifier to upper case
[INFO] [stderr]    |
[INFO] [stderr] 20 -         const data_len: usize = MIN_SIZE+1;
[INFO] [stderr] 20 +         const DATA_LEN: usize = MIN_SIZE+1;
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: constant `data_len` should have an upper case name
[INFO] [stderr]   --> src/predictor/taylor_approx.rs:33:15
[INFO] [stderr]    |
[INFO] [stderr] 33 |         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stderr]    |               ^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: convert the identifier to upper case
[INFO] [stderr]    |
[INFO] [stderr] 33 -         const data_len: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stderr] 33 +         const DATA_LEN: usize = TAYLOR_POLY_ORDER as usize+1;
[INFO] [stderr]    |
[INFO] [stderr] 
[INFO] [stderr] warning: `neuralink_final` (lib) generated 9 warnings (run `cargo fix --lib -p neuralink_final` to apply 5 suggestions)
[INFO] [stderr] warning: `neuralink_final` (lib test) generated 9 warnings (9 duplicates)
[INFO] [stderr] warning: unused import: `predictor::taylor_approx::TaylorQuadraticApproximator`
[INFO] [stderr]   --> src/main.rs:12:5
[INFO] [stderr]    |
[INFO] [stderr] 12 | use predictor::taylor_approx::TaylorQuadraticApproximator;
[INFO] [stderr]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: unused import: `predictor::oracle_approx::OraclePredictor`
[INFO] [stderr]   --> src/main.rs:14:5
[INFO] [stderr]    |
[INFO] [stderr] 14 | use predictor::oracle_approx::OraclePredictor;
[INFO] [stderr]    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: field `msg` is never read
[INFO] [stderr]  --> src/interface.rs:7:26
[INFO] [stderr]   |
[INFO] [stderr] 7 |     CommunicationError { msg: String },
[INFO] [stderr]   |     ------------------   ^^^
[INFO] [stderr]   |     |
[INFO] [stderr]   |     field in this variant
[INFO] [stderr]   |
[INFO] [stderr]   = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stderr]   = note: `#[warn(dead_code)]` on by default
[INFO] [stderr] 
[INFO] [stderr] warning: variants `AcquisitionError`, `TimeoutError`, and `PredictionError` are never constructed
[INFO] [stderr]   --> src/interface.rs:5:5
[INFO] [stderr]    |
[INFO] [stderr] 3  | pub enum OCTError {
[INFO] [stderr]    |          -------- variants in this enum
[INFO] [stderr] 4  |     // Failed to acquire data from the OCT laser
[INFO] [stderr] 5  |     AcquisitionError { msg: String },
[INFO] [stderr]    |     ^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 9  |     TimeoutError { msg: String },
[INFO] [stderr]    |     ^^^^^^^^^^^^
[INFO] [stderr] 10 |
[INFO] [stderr] 11 |     PredictionError { msg: String },
[INFO] [stderr]    |     ^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `OCTError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stderr] 
[INFO] [stderr] warning: field `msg` is never read
[INFO] [stderr]   --> src/interface.rs:56:17
[INFO] [stderr]    |
[INFO] [stderr] 56 |     MoveError { msg: String },
[INFO] [stderr]    |     ---------   ^^^
[INFO] [stderr]    |     |
[INFO] [stderr]    |     field in this variant
[INFO] [stderr]    |
[INFO] [stderr]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stderr] 
[INFO] [stderr] warning: variants `ConnectionError` and `PositionError` are never constructed
[INFO] [stderr]   --> src/interface.rs:58:5
[INFO] [stderr]    |
[INFO] [stderr] 54 | pub enum RobotError {
[INFO] [stderr]    |          ---------- variants in this enum
[INFO] [stderr] ...
[INFO] [stderr] 58 |     ConnectionError { msg: String },
[INFO] [stderr]    |     ^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 61 |     PositionError { msg: String },
[INFO] [stderr]    |     ^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `RobotError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stderr] 
[INFO] [stderr] warning: method `command_grasp` is never used
[INFO] [stderr]   --> src/interface.rs:75:14
[INFO] [stderr]    |
[INFO] [stderr] 71 | pub trait Robot {
[INFO] [stderr]    |           ----- method in this trait
[INFO] [stderr] ...
[INFO] [stderr] 75 |     async fn command_grasp(&self) -> Result<(), RobotError>;
[INFO] [stderr]    |              ^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: field `state_errors` is never read
[INFO] [stderr]   --> src/robot.rs:15:9
[INFO] [stderr]    |
[INFO] [stderr] 13 | pub struct RobotArm {
[INFO] [stderr]    |            -------- field in this struct
[INFO] [stderr] 14 |     pub distance_errors: bool,
[INFO] [stderr] 15 |     pub state_errors: bool,
[INFO] [stderr]    |         ^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: method `train` is never used
[INFO] [stderr]   --> src/predictor/mod.rs:10:8
[INFO] [stderr]    |
[INFO] [stderr] 8  | pub trait BrainPredictor {
[INFO] [stderr]    |           -------------- method in this trait
[INFO] [stderr] 9  |     fn predict(&self, distances: &Vec<Result<u64, OCTError>>, times: &Vec<Instant>, print_coefs: bool) -> Option<impl Fn(f64) -> f64>;
[INFO] [stderr] 10 |     fn train(&self) -> bool{
[INFO] [stderr]    |        ^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: constant `MIN_SIZE` is never used
[INFO] [stderr]  --> src/predictor/oracle_approx.rs:5:7
[INFO] [stderr]   |
[INFO] [stderr] 5 | const MIN_SIZE: usize =3;
[INFO] [stderr]   |       ^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stderr]  --> src/predictor/oracle_approx.rs:6:7
[INFO] [stderr]   |
[INFO] [stderr] 6 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stderr]   |       ^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: struct `OraclePredictor` is never constructed
[INFO] [stderr]  --> src/predictor/oracle_approx.rs:8:12
[INFO] [stderr]   |
[INFO] [stderr] 8 | pub struct OraclePredictor{
[INFO] [stderr]   |            ^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: associated functions `new` and `passes_predict_assumptions` are never used
[INFO] [stderr]   --> src/predictor/oracle_approx.rs:13:12
[INFO] [stderr]    |
[INFO] [stderr] 12 | impl OraclePredictor{
[INFO] [stderr]    | -------------------- associated functions in this implementation
[INFO] [stderr] 13 |     pub fn new() -> OraclePredictor{
[INFO] [stderr]    |            ^^^
[INFO] [stderr] ...
[INFO] [stderr] 19 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(Vec<u64>, Vec<Instan...
[INFO] [stderr]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: constant `MAX_LATENCY_MS` is never used
[INFO] [stderr]  --> src/predictor/taylor_approx.rs:4:7
[INFO] [stderr]   |
[INFO] [stderr] 4 | const MAX_LATENCY_MS: u64 = 18;
[INFO] [stderr]   |       ^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: constant `MAX_LATENCY_STD_MS` is never used
[INFO] [stderr]  --> src/predictor/taylor_approx.rs:5:7
[INFO] [stderr]   |
[INFO] [stderr] 5 | const MAX_LATENCY_STD_MS: u64 = 3;
[INFO] [stderr]   |       ^^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: constant `TAYLOR_POLY_ORDER` is never used
[INFO] [stderr]  --> src/predictor/taylor_approx.rs:6:7
[INFO] [stderr]   |
[INFO] [stderr] 6 | const TAYLOR_POLY_ORDER: u64 = 2; 
[INFO] [stderr]   |       ^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: struct `TaylorQuadraticApproximator` is never constructed
[INFO] [stderr]  --> src/predictor/taylor_approx.rs:8:12
[INFO] [stderr]   |
[INFO] [stderr] 8 | pub struct TaylorQuadraticApproximator;
[INFO] [stderr]   |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: associated function `passes_predict_assumptions` is never used
[INFO] [stderr]   --> src/predictor/taylor_approx.rs:32:8
[INFO] [stderr]    |
[INFO] [stderr] 10 | impl TaylorQuadraticApproximator{
[INFO] [stderr]    | -------------------------------- associated function in this implementation
[INFO] [stderr] ...
[INFO] [stderr] 32 |     fn passes_predict_assumptions(distance_queue: &Vec<Result<u64, OCTError>>, time_queue: &Vec<Instant>) -> Result<(f64, f64, Vec<u64>, ...
[INFO] [stderr]    |        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] 
[INFO] [stderr] warning: `neuralink_final` (bin "neuralink_final") generated 23 warnings (5 duplicates) (run `cargo fix --bin "neuralink_final"` to apply 2 suggestions)
[INFO] [stderr] warning: `neuralink_final` (bin "neuralink_final" test) generated 23 warnings (23 duplicates)
[INFO] [stderr]     Finished `test` profile [unoptimized + debuginfo] target(s) in 0.13s
[INFO] [stderr]      Running unittests src/lib.rs (/opt/rustwide/target/debug/deps/neuralink_final-ad99ee576b70c734)
[INFO] [stdout] 
[INFO] [stdout] running 0 tests
[INFO] [stdout] 
[INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
[INFO] [stdout] 
[INFO] [stderr]      Running unittests src/main.rs (/opt/rustwide/target/debug/deps/neuralink_final-3d965dc64f01010d)
[INFO] [stdout] 
[INFO] [stdout] running 0 tests
[INFO] [stdout] 
[INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
[INFO] [stdout] 
[INFO] [stderr]      Running tests/integration_tests_oracle.rs (/opt/rustwide/target/debug/deps/integration_tests_oracle-5456287bb1a61646)
[INFO] [stdout] 
[INFO] [stdout] running 3 tests
[INFO] [stdout] test test_controller_distance_errors_oracle has been running for over 60 seconds
[INFO] [stdout] test test_controller_move_errors_oracle has been running for over 60 seconds
[INFO] [stdout] test test_controller_no_errors_oracle has been running for over 60 seconds
[INFO] [stdout] test test_controller_no_errors_oracle ... FAILED
[INFO] [stdout] test test_controller_distance_errors_oracle ... FAILED
[ERROR] error running command: no output for 300 seconds
[INFO] running `Command { std: "docker" "inspect" "37166713a04c70ff33a1eb12a80fc0babee90e1d6e636e28a8428799ac80f934", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "37166713a04c70ff33a1eb12a80fc0babee90e1d6e636e28a8428799ac80f934", kill_on_drop: false }`
[INFO] [stdout] 37166713a04c70ff33a1eb12a80fc0babee90e1d6e636e28a8428799ac80f934
