databio / gtars

Performance-critical tools to manipulate, analyze, and process genomic interval data. Primarily focused on building tools for geniml - our genomic machine learning python package.
3 stars 2 forks source link

cargo build causes error on newest branch dev_bam_bedgraph_bigtools #46

Closed ClaudeHu closed 3 weeks ago

ClaudeHu commented 3 weeks ago
(text2bed) -bash-4.4$cd /home/zh4nh/repo/gtars/gtars
(text2bed) -bash-4.4$cargo build
   Compiling gtars v0.0.15 (/sfs/gpfs/tardis/home/zh4nh/repo/gtars/gtars)
error[E0560]: struct `BBIWriteArgs` has no field named `zooms`
   --> src/uniwig/writing.rs:203:17
    |
203 |                 zooms: None,
    |                 ^^^^^ `BBIWriteArgs` does not have this field
    |
    = note: all struct fields are already assigned

warning: unused variable: `flags`
   --> src/uniwig/reading.rs:336:13
    |
336 |         let flags = record.flags();
    |             ^^^^^ help: if this is intentional, prefix it with an underscore: `_flags`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: unused variable: `stepsize`
   --> src/uniwig/writing.rs:128:5
    |
128 |     stepsize: i32,
    |     ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_stepsize`

warning: unused variable: `i`
  --> src/uniwig/utils.rs:20:13
   |
20 |     for (u, i) in count_results.0.iter().zip(count_results.1.iter()) {
   |             ^ help: if this is intentional, prefix it with an underscore: `_i`

warning: variable does not need to be mutable
  --> src/uniwig/utils.rs:14:9
   |
14 |     let mut previous_start = start_position as u32;
   |         ----^^^^^^^^^^^^^^
   |         |
   |         help: remove this `mut`
   |
   = note: `#[warn(unused_mut)]` on by default

For more information about this error, try `rustc --explain E0560`.
warning: `gtars` (lib) generated 4 warnings
error: could not compile `gtars` (lib) due to 1 previous error; 4 warnings emitted
nleroy917 commented 3 weeks ago

I cant reproduce this mine compiles fine. Have you pulled updates and what are you running?

ClaudeHu commented 3 weeks ago

I cant reproduce this mine compiles fine. Have you pulled updates and what are you running?

yes I ran git checkout and git pull which shows already up to date

(text2bed) -bash-4.4$git checkout dev_bam_bedgraph_bigtools
Already on 'dev_bam_bedgraph_bigtools'
Your branch is up to date with 'origin/dev_bam_bedgraph_bigtools'.
(text2bed) -bash-4.4$git pull
Enter passphrase for key '/home/zh4nh/.ssh/id_ed25519': 
Already up to date.
(text2bed) -bash-4.4$cd gtars
(text2bed) -bash-4.4$cargo build
   Compiling gtars v0.0.15 (/sfs/gpfs/tardis/home/zh4nh/repo/gtars/gtars)
error[E0560]: struct `BBIWriteArgs` has no field named `zooms`
   --> src/uniwig/writing.rs:203:17
    |
203 |                 zooms: None,
    |                 ^^^^^ `BBIWriteArgs` does not have this field
    |
    = note: all struct fields are already assigned

warning: unused variable: `flags`
   --> src/uniwig/reading.rs:336:13
    |
336 |         let flags = record.flags();
    |             ^^^^^ help: if this is intentional, prefix it with an underscore: `_flags`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: unused variable: `stepsize`
   --> src/uniwig/writing.rs:128:5
    |
128 |     stepsize: i32,
    |     ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_stepsize`

warning: unused variable: `i`
  --> src/uniwig/utils.rs:20:13
   |
20 |     for (u, i) in count_results.0.iter().zip(count_results.1.iter()) {
   |             ^ help: if this is intentional, prefix it with an underscore: `_i`

warning: variable does not need to be mutable
  --> src/uniwig/utils.rs:14:9
   |
14 |     let mut previous_start = start_position as u32;
   |         ----^^^^^^^^^^^^^^
   |         |
   |         help: remove this `mut`
   |
   = note: `#[warn(unused_mut)]` on by default

For more information about this error, try `rustc --explain E0560`.
warning: `gtars` (lib) generated 4 warnings
error: could not compile `gtars` (lib) due to 1 previous error; 4 warnings emitted
ClaudeHu commented 3 weeks ago

That error is gone after I replaced the old repo with a fresh clone.