Skip to content

Added Deep AI (handles forms properly)#31

Draft
Mitchell-UCF-Git wants to merge 1 commit intomainfrom
add-form-handling
Draft

Added Deep AI (handles forms properly)#31
Mitchell-UCF-Git wants to merge 1 commit intomainfrom
add-form-handling

Conversation

@Mitchell-UCF-Git
Copy link
Contributor

added the working yaml files from Deep AI and created a folder to hold all of our additional files.

Copilot AI review requested due to automatic review settings March 26, 2025 18:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds YAML configuration files for image generation workflows and parameters, supporting both Dall‑E and Deep AI APIs.

  • Adds workflow configuration (imageWorkflow.yaml) for Dall‑E with API details
  • Adds image generation parameter configuration (imagegen.yaml) for Dall‑E
  • Adds corresponding Deep AI configuration files for workflow and image generation

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
additional_models/Dall-E/imageWorkflow.yaml Workflow config for Dall‑E image generation
additional_models/Dall-E/imagegen.yaml Parameter config for Dall‑E image generation
additional_models/deep_ai/imageWorkflow.yaml Workflow config for Deep AI image generation
additional_models/deep_ai/imagegen.yaml Parameter config for Deep AI image generation

response_path: "data.url"
response_format: "url"
file_id_path: "created"
file_extention: "png"
Copy link

Copilot AI Mar 26, 2025

Choose a reason for hiding this comment

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

Typo in the key 'file_extention'; consider renaming it to 'file_extension' to improve clarity and consistency.

Suggested change
file_extention: "png"
file_extension: "png"

Copilot uses AI. Check for mistakes.
response_path: "output_url"
response_format: "url"
file_id_path: "id"
file_extention: "jpg"
Copy link

Copilot AI Mar 26, 2025

Choose a reason for hiding this comment

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

Typo in the key 'file_extention'; consider renaming it to 'file_extension' to ensure consistency.

Suggested change
file_extention: "jpg"
file_extension: "jpg"

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +2
#Paramaters for OpenAI's Image Generation API
#The APIConfig struct assumes that each required and optional paramater has a description
Copy link

Copilot AI Mar 26, 2025

Choose a reason for hiding this comment

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

Typo in the comment; 'Paramaters' should be corrected to 'Parameters'.

Suggested change
#Paramaters for OpenAI's Image Generation API
#The APIConfig struct assumes that each required and optional paramater has a description
#Parameters for OpenAI's Image Generation API
#The APIConfig struct assumes that each required and optional parameter has a description

Copilot uses AI. Check for mistakes.
Comment on lines +1 to +2
#Paramaters for OpenAI's Image Generation API
#The APIConfig struct assumes that each required and optional paramater has a description
Copy link

Copilot AI Mar 26, 2025

Choose a reason for hiding this comment

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

Typo in the comment; 'paramater' should be corrected to 'parameter'.

Suggested change
#Paramaters for OpenAI's Image Generation API
#The APIConfig struct assumes that each required and optional paramater has a description
#Parameters for OpenAI's Image Generation API
#The APIConfig struct assumes that each required and optional parameter has a description

Copilot uses AI. Check for mistakes.
size:
default: "1024x1024"
description: "The size of the generated images."
options: ["256x256", "512x512", "1024x1024", "1024x1024", "1792x1024", "1024x1792"]
Copy link

Copilot AI Mar 26, 2025

Choose a reason for hiding this comment

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

The '1024x1024' size option is duplicated. Removing the duplicate will prevent potential confusion and ensure correct configuration.

Suggested change
options: ["256x256", "512x512", "1024x1024", "1024x1024", "1792x1024", "1024x1792"]
options: ["256x256", "512x512", "1024x1024", "1792x1024", "1024x1792"]

Copilot uses AI. Check for mistakes.
@Jacob-Stella Jacob-Stella marked this pull request as draft March 26, 2025 21:29
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.

1 participant