Skip to content

Commit c84410b

Browse files
author
Jicheng Lu
committed
fix file selection
1 parent adfb8f4 commit c84410b

File tree

5 files changed

+13
-24
lines changed

5 files changed

+13
-24
lines changed

src/Infrastructure/BotSharp.Abstraction/Files/Models/SelectFileOptions.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ public class SelectFileOptions
88
public string? Provider { get; set; }
99

1010
/// <summary>
11-
/// Llm model id
11+
/// Llm model
1212
/// </summary>
13-
public string? ModelId { get; set; }
13+
public string? Model { get; set; }
1414

1515
/// <summary>
1616
/// Agent id

src/Infrastructure/BotSharp.Core/Files/Services/Instruct/FileInstructService.Image.cs

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public async Task<RoleDialogModel> GenerateImage(string text, InstructOptions? o
5757
{
5858
Id = innerAgentId,
5959
Instruction = instruction
60-
}, new RoleDialogModel(AgentRole.User, text));
60+
}, new RoleDialogModel(AgentRole.User, instruction ?? text));
6161

6262
var hooks = _services.GetServices<IInstructHook>();
6363
foreach (var hook in hooks)
@@ -90,8 +90,6 @@ public async Task<RoleDialogModel> VaryImage(InstructFileModel image, InstructOp
9090
}
9191

9292
var innerAgentId = options?.AgentId ?? Guid.Empty.ToString();
93-
var instruction = await GetAgentTemplate(innerAgentId, options?.TemplateName);
94-
9593
var completion = CompletionProvider.GetImageCompletion(_services, provider: options?.Provider ?? "openai", model: options?.Model ?? "dall-e-2");
9694
var bytes = await DownloadFile(image);
9795
using var stream = new MemoryStream();
@@ -101,8 +99,7 @@ public async Task<RoleDialogModel> VaryImage(InstructFileModel image, InstructOp
10199
var fileName = $"{image.FileName ?? "image"}.{image.FileExtension ?? "png"}";
102100
var message = await completion.GetImageVariation(new Agent()
103101
{
104-
Id = innerAgentId,
105-
Instruction = instruction
102+
Id = innerAgentId
106103
}, new RoleDialogModel(AgentRole.User, string.Empty), stream, fileName);
107104

108105
stream.Close();
@@ -120,9 +117,7 @@ await hook.OnResponseGenerated(new InstructResponseModel
120117
AgentId = innerAgentId,
121118
Provider = completion.Provider,
122119
Model = completion.Model,
123-
TemplateName = options?.TemplateName,
124120
UserMessage = string.Empty,
125-
SystemInstruction = instruction,
126121
CompletionText = message.Content
127122
});
128123
}
@@ -149,9 +144,8 @@ public async Task<RoleDialogModel> EditImage(string text, InstructFileModel imag
149144
var fileName = $"{image.FileName ?? "image"}.{image.FileExtension ?? "png"}";
150145
var message = await completion.GetImageEdits(new Agent()
151146
{
152-
Id = innerAgentId,
153-
Instruction = instruction
154-
}, new RoleDialogModel(AgentRole.User, text), stream, fileName);
147+
Id = innerAgentId
148+
}, new RoleDialogModel(AgentRole.User, instruction ?? text), stream, fileName);
155149

156150
stream.Close();
157151

@@ -205,9 +199,8 @@ public async Task<RoleDialogModel> EditImage(string text, InstructFileModel imag
205199
var maskName = $"{mask.FileName ?? "mask"}.{mask.FileExtension ?? "png"}";
206200
var message = await completion.GetImageEdits(new Agent()
207201
{
208-
Id = innerAgentId,
209-
Instruction = instruction
210-
}, new RoleDialogModel(AgentRole.User, text), imageStream, imageName, maskStream, maskName);
202+
Id = innerAgentId
203+
}, new RoleDialogModel(AgentRole.User, instruction ?? text), imageStream, imageName, maskStream, maskName);
211204

212205
imageStream.Close();
213206
maskStream.Close();

src/Infrastructure/BotSharp.Core/Files/Services/Instruct/FileInstructService.SelectFile.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,10 +93,9 @@ private async Task<IEnumerable<MessageFileModel>> SelectFiles(IEnumerable<Messag
9393
}
9494

9595
var providerName = options.Provider ?? "openai";
96-
var modelId = options?.ModelId ?? "gpt-4o";
96+
var model = options?.Model ?? "gpt-4o-mini";
9797
var provider = llmProviderService.GetProviders().FirstOrDefault(x => x == providerName);
98-
var model = llmProviderService.GetProviderModel(provider: provider, id: modelId);
99-
var completion = CompletionProvider.GetChatCompletion(_services, provider: provider, model: model.Name);
98+
var completion = CompletionProvider.GetChatCompletion(_services, provider: provider, model: model);
10099

101100
var response = await completion.GetChatCompletions(agent, new List<RoleDialogModel> { message });
102101
var content = response?.Content ?? string.Empty;

src/Infrastructure/BotSharp.OpenAPI/Controllers/InstructModeController.cs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -242,8 +242,7 @@ public async Task<ImageGenerationViewModel> ImageVariation([FromBody] ImageVaria
242242
{
243243
Provider = input.Provider,
244244
Model = input.Model,
245-
AgentId = input.AgentId,
246-
TemplateName = input.TemplateName
245+
AgentId = input.AgentId
247246
});
248247
imageViewModel.Content = message.Content;
249248
imageViewModel.Images = message.GeneratedImages.Select(x => ImageViewModel.ToViewModel(x)).ToList();
@@ -262,7 +261,7 @@ public async Task<ImageGenerationViewModel> ImageVariation([FromBody] ImageVaria
262261
[HttpPost("/instruct/image-variation/upload")]
263262
public async Task<ImageGenerationViewModel> ImageVariation(IFormFile file, [FromForm] string? provider = null,
264263
[FromForm] string? model = null, [FromForm] List<MessageState>? states = null,
265-
[FromForm] string? agentId = null, [FromForm] string? templateName = null)
264+
[FromForm] string? agentId = null)
266265
{
267266
var state = _services.GetRequiredService<IConversationStateService>();
268267
states?.ForEach(x => state.SetState(x.Key, x.Value, activeRounds: x.ActiveRounds, source: StateSource.External));
@@ -276,8 +275,7 @@ public async Task<ImageGenerationViewModel> ImageVariation(IFormFile file, [From
276275
{
277276
Provider = provider,
278277
Model = model,
279-
AgentId = agentId,
280-
TemplateName = templateName
278+
AgentId = agentId
281279
});
282280

283281
imageViewModel.Content = message.Content;

src/Plugins/BotSharp.Plugin.OpenAI/Providers/Image/ImageCompletionProvider.Edit.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using OpenAI.Images;
2-
using static System.Net.Mime.MediaTypeNames;
32

43
namespace BotSharp.Plugin.OpenAI.Providers.Image;
54

0 commit comments

Comments
 (0)