Skip to content

HuggingFaceM4/Idefics3-8B-Llama3 crash fix #3267

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions router/src/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -265,6 +265,10 @@ impl Idefics3 {
pub fn get_max_longest_edge_for_image_resize(&self) -> usize {
1456
}

pub fn get_max_image_size(&self) -> usize {
4096
}
}

#[derive(Clone, Debug, Serialize, Deserialize)]
Expand Down
26 changes: 20 additions & 6 deletions router/src/validation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -646,11 +646,10 @@ fn image_tokens(
const GLOBAL_IMG: &str = "<global-img>";

let max_longest_edge_for_image_resize = config.get_max_longest_edge_for_image_resize();
let max_image_size = config.get_max_image_size();

// resize image if it is larger than max_longest_edge_for_image_resize keeping aspect ratio
let (height, width) = if height > max_longest_edge_for_image_resize
|| width > max_longest_edge_for_image_resize
{
// resize image to max_longest_edge_for_image_resize and keep aspect ratio
let (height, width) = {
let aspect_ratio = height as f32 / width as f32;
if height > width {
(
Expand All @@ -663,8 +662,23 @@ fn image_tokens(
max_longest_edge_for_image_resize,
)
}
} else {
(height, width)
};

let (height, width) = {
let aspect_ratio = height as f32 / width as f32;
if height >= width && height > max_image_size {
(
max_image_size,
(max_image_size as f32 / aspect_ratio) as usize,
)
} else if width > height && width > max_image_size {
(
(max_image_size as f32 * aspect_ratio) as usize,
max_image_size,
)
} else {
(height, width)
}
};

let image_seq_len = config.get_number_of_features();
Expand Down