Skip to content

Commit 5c8ccfa

Browse files
committed
Working on vision chat templates
1 parent 8cdc4a0 commit 5c8ccfa

File tree

3 files changed

+70
-44
lines changed

3 files changed

+70
-44
lines changed

Libraries/MLXVLM/Models/Qwen2VL.swift

Lines changed: 37 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -686,53 +686,60 @@ public class Qwen2VLProcessor: UserInputProcessor {
686686
return (flattenedPatches, .init(gridT, gridH, gridW))
687687
}
688688

689-
public func prepare(prompt: UserInput.Prompt, imageTHW: [THW]?) -> String {
690-
// the tokenizer does have a chat template and it expects messages
691-
// like this:
689+
public func prepare(prompt: UserInput.Prompt, imageTHW: [THW]?) throws -> [Int] {
690+
// The tokenizer has a chat template and expects messages like this:
692691
//
693692
// [{'role': 'user', 'content': [{'type': 'text', 'text': 'What are these?'},
694693
// {'type': 'image'}, {'type': 'image'}, {'type': 'image'}]}]
695694
//
696695
// The output of the prompt template is fed into
697696
// image_processing_qwen2_vl.preprocess where it is further augmented
698697
// by replacing tokens according to imageTHW.
699-
//
700-
// Neither the structured content nor the postprocessing of the template
701-
// are supported in current Tokenizer/Jinja (swift) so handle that here.
702-
703-
var messages = prompt.asMessages()
704-
if messages[0]["role"] != "system" {
705-
messages.insert(["role": "system", "content": "You are a helpful assistant."], at: 0)
706-
}
707698

708-
let lastIndex = messages.count - 1
709-
var lastMessage = messages[lastIndex]["content"] ?? ""
699+
var messages = {
700+
switch prompt {
701+
case .messages(let messages):
702+
// TODO: Handle this case
703+
// ...
704+
var messages = messages
705+
if messages[0]["role"] != "system" {
706+
messages.insert(
707+
["role": "system", "content": "You are a helpful assistant."], at: 0)
708+
}
709+
return messages as [[String: Any]]
710+
case .text(let text):
711+
return [
712+
[
713+
"role": "user",
714+
"content": [
715+
[
716+
"type": "text",
717+
"text": text,
718+
],
719+
[
720+
"type": "image",
721+
"image": "base64_encoded_image_data", // TODO: Image data goes here?
722+
],
723+
],
724+
]
725+
] as [[String: Any]]
726+
}
727+
}()
710728

711-
// image_processing_qwen2_vl.preprocess -- inject image_pad tokens for each image
712-
let mergeLength = config.mergeSize * config.mergeSize
713729
for thw in imageTHW ?? [] {
714-
lastMessage += "<|vision_start|>"
715-
lastMessage += Array(repeating: "<|image_pad|>", count: thw.product / mergeLength)
716-
.joined()
717-
lastMessage += "<|vision_end|>"
730+
// TODO: Add images to messages
731+
// ...
718732
}
719733

720-
messages[lastIndex]["content"] = lastMessage
734+
print(messages)
721735

722-
return
723-
messages
724-
.map {
725-
"<|im_start|>\($0["role"] ?? "user")\n\($0["content"] ?? "")<|im_end|>"
726-
}
727-
.joined(separator: "\n")
728-
+ "\n<|im_start|>assistant\n"
736+
return try tokenizer.applyChatTemplate(messages: messages)
729737
}
730738

731739
public func prepare(input: UserInput) throws -> LMInput {
732740
if input.images.isEmpty {
733741
// just a straight text prompt
734-
let prompt = prepare(prompt: input.prompt, imageTHW: nil)
735-
let promptTokens = try tokenizer.encode(text: prompt)
742+
let promptTokens = try prepare(prompt: input.prompt, imageTHW: nil)
736743
return LMInput(tokens: MLXArray(promptTokens))
737744
}
738745

@@ -744,8 +751,7 @@ public class Qwen2VLProcessor: UserInputProcessor {
744751
let image = LMInput.ProcessedImage(pixels: pixels, imageGridThw: images.map { $0.1 })
745752

746753
// processing_qwen2_vl.Qwen2VLProcessor
747-
let prompt = prepare(prompt: input.prompt, imageTHW: image.imageGridThw)
748-
let promptTokens = try tokenizer.encode(text: prompt)
754+
let promptTokens = try prepare(prompt: input.prompt, imageTHW: image.imageGridThw)
749755
let promptArray = MLXArray(promptTokens).expandedDimensions(axis: 0)
750756
let mask = ones(like: promptArray).asType(.int8)
751757

Package.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ let package = Package(
2828
],
2929
dependencies: [
3030
.package(url: "https://github.com/ml-explore/mlx-swift", from: "0.21.2"),
31-
.package(url: "https://github.com/huggingface/swift-transformers", from: "0.1.13"),
31+
// .package(url: "https://github.com/huggingface/swift-transformers", from: "0.1.13"),
32+
.package(
33+
url: "https://github.com/DePasqualeOrg/swift-transformers", branch: "images-and-tools"),
3234
.package(url: "https://github.com/1024jp/GzipSwift", "6.0.1" ... "6.0.1"),
3335
.package(url: "https://github.com/apple/swift-async-algorithms", from: "1.0.0"),
3436
],

mlx-swift-examples.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved

Lines changed: 30 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)