Skip to content

Goal refactor to allow for local image files#110

Merged
gasse merged 42 commits intomainfrom
image_download
Oct 8, 2024
Merged

Goal refactor to allow for local image files#110
gasse merged 42 commits intomainfrom
image_download

Conversation

@ljang0
Copy link
Copy Markdown
Collaborator

@ljang0 ljang0 commented Aug 9, 2024

@ThibaultLSDC can you check this?

@ljang0 ljang0 changed the title test code Downloading Images to TMP file when Task is Instantiated Aug 9, 2024
@gasse gasse changed the title Downloading Images to TMP file when Task is Instantiated Goal refactor to allow for local image files Sep 19, 2024
@gasse gasse requested a review from TLSDC September 19, 2024 19:59
Copy link
Copy Markdown
Collaborator

@recursix recursix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job, But I'm really disappointed in the number of unit tests :p you will regret it.

recursix
recursix previously approved these changes Sep 25, 2024
}
)
elif msg["role"] == "user_image":
system_msgs.append({"type": "image_url", "image_url": msg["message"]})
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
system_msgs.append({"type": "image_url", "image_url": msg["message"]})
user_msgs.append({"type": "image_url", "image_url": msg["message"]})

?

@gasse gasse merged commit 9dd62dd into main Oct 8, 2024
@gasse gasse deleted the image_download branch October 8, 2024 15:12
layahaasini pushed a commit to layahaasini/BrowserGym that referenced this pull request Nov 21, 2025
---------

Co-authored-by: Thibault Le Sellier de Chezelles <thibault.de.chezelles@gmail.com>
Co-authored-by: Maxime Gasse <maxime.gasse@gmail.com>
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.

4 participants