Skip to content

Commit

Permalink
Merge pull request #44 from pangenome/avoid_short_chunks
Browse files Browse the repository at this point in the history
max_depth as u16 to reduce memory usage
  • Loading branch information
AndreaGuarracino authored Jan 30, 2025
2 parents 6d5db05 + a33132d commit a620f08
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions src/impg.rs
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ impl Impg {
range_start: i32,
range_end: i32,
masked_regions: Option<&FxHashMap<u32, SortedRanges>>,
max_depth: i32,
max_depth: u16,
min_transitive_region_size: i32,
min_distance_between_ranges: i32,
) -> Vec<AdjustedInterval> {
Expand All @@ -416,7 +416,7 @@ impl Impg {
}
));
// Initialize stack with first query
let mut stack = vec![(target_id, range_start, range_end, 0i32)];
let mut stack = vec![(target_id, range_start, range_end, 0u16)];
// Initialize visited ranges from masked regions if provided
let mut visited_ranges: FxHashMap<u32, SortedRanges> = if let Some(m) = masked_regions {
m.iter()
Expand Down
6 changes: 3 additions & 3 deletions src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ enum Args {

/// Maximum recursion depth for transitive overlaps (0 for no limit)
#[clap(short = 'm', long, value_parser, default_value_t = 1)]
max_depth: i32,
max_depth: u16,

/// Minimum size of intervals to consider for transitive queries
#[clap(long, value_parser, default_value_t = 500)]
Expand Down Expand Up @@ -91,7 +91,7 @@ enum Args {

/// Maximum recursion depth for transitive overlaps (0 for no limit)
#[clap(short = 'm', long, value_parser, default_value_t = 1)]
max_depth: i32,
max_depth: u16,

/// Minimum size of intervals to consider for transitive queries
#[clap(long, value_parser, default_value_t = 0)]
Expand Down Expand Up @@ -330,7 +330,7 @@ fn perform_query(
target_name: &str,
target_range: (i32, i32),
transitive: bool,
max_depth: i32,
max_depth: u16,
min_transitive_region_size: i32,
min_distance_between_ranges: i32
) -> Vec<AdjustedInterval> {
Expand Down
2 changes: 1 addition & 1 deletion src/partition.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ pub fn partition_alignments(
sequence_prefix: &str,
min_region_size: i32,
merge_distance: i32,
max_depth: i32,
max_depth: u16,
min_transitive_region_size: i32,
min_distance_between_ranges: i32,
min_mask_proximity: i32,
Expand Down

0 comments on commit a620f08

Please sign in to comment.