Skip to content

1.2#9

Merged
MrGKanev merged 6 commits into
masterfrom
1.2
Oct 25, 2025
Merged

1.2#9
MrGKanev merged 6 commits into
masterfrom
1.2

Conversation

@MrGKanev

Copy link
Copy Markdown
Owner

No description provided.

@MrGKanev MrGKanev requested a review from Copilot September 19, 2025 13:08

Copilot AI left a comment

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.

Pull Request Overview

This PR updates the project to version 1.2 with a focus on cleaning up emoji characters from output messages and restructuring the build process. The changes improve text readability and simplify the Docker build configuration.

  • Removes emoji characters from console output messages across multiple files
  • Updates version number from 1.1 to 1.2 in Docker metadata
  • Restructures Docker build to use requirements.txt instead of inline pip installs

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated 1 comment.

File Description
test_gpu.py Completely removed - file deleted
scripts/tf_benchmark.py Removed emoji characters from console print statements
entrypoint.sh Removed emoji characters from echo statements and warning messages
Dockerfile.gpu Updated version label and restructured package installation to use requirements.txt
Comments suppressed due to low confidence (1)

scripts/tf_benchmark.py:1

  • Missing character in the information icon. Should be 'ℹ️' instead of 'ℹ'.
#!/usr/bin/env python3

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread entrypoint.sh Outdated
MrGKanev and others added 3 commits September 19, 2025 16:08
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@MrGKanev MrGKanev merged commit de13e6d into master Oct 25, 2025
1 check passed
@MrGKanev MrGKanev deleted the 1.2 branch October 25, 2025 12:57
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.

2 participants