Skip to content

Xavier and Tim Pull Request #43

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
xavierb117 wants to merge 142 commits into
base: main
Choose a base branch
from
Open

Xavier and Tim Pull Request #43

xavierb117 wants to merge 142 commits into from

Conversation

@xavierb117
Copy link

@xavierb117 xavierb117 commented Nov 18, 2025

No description provided.

xavierb117 and others added 30 commits October 8, 2025 23:02
@xavierb117
I, xavierb117, understand that AI is ONLY to be used for tests, and t…
78a5786 
…hat every commit that I use AI for must start with 'AI Used'
@timmyd4
I, timmyd4, understand that AI is ONLY to be used for tests, and that…
541ab1f 
… every commit that I use AI for must start with 'AI Used'
@timmyd4
Added some notes
5a4ff7d
@xavierb117
Added more notes for the input, output, and imagesummaryapp
4b73756
@xavierb117
Made notes on DfsBinaryGroupFinder
dc0ee7e
@timmyd4
EuclideanColorDistance notes
92efcd7
@xavierb117
Wave 1 - Potentially finished taking notes on input, output, classes,…
52610d8 
… records, and interfaces
@timmyd4
Added exceptions
e9e1c9a
@xavierb117
AI Used - Created the DFSBinaryGroupFinderTest file and created initi…
ba86ce7 
…al set of tests by AI
@xavierb117
Wave 2 - Initial solution for DfsBinaryGroupFinder
b916e85
@timmyd4
working on EuclideanColorDistance
2156497
@timmyd4
fixed maybe?
e749a1e
@timmyd4
mistake
740b0e1
@xavierb117
Merge remote-tracking branch 'upstream/main'
fd54db5
@xavierb117
AI Used - Used AI to create test cases for EuclideanColorDistancec
2eb8329
@xavierb117
Replaced decimal point with actual Math.sqrt to better confirm the ex…
655fa8e 
…pected value
@timmyd4
AI Used Tests Start for DistanceImageBinarizer
1aa9074
@timmyd4
passed all inital tests for toBinaryArray - may think of more
0f081f3
@timmyd4
Added another test without AI!
31ec68a
@timmyd4
Added another test without AI
f8708a7
@timmyd4
Finished toBufferdImage method - basically same logic as toBinaryArray
6198601
@xavierb117
Forgot to push the modified expected value to full value and not Math…
efb6633 
….sqrt
@elena5100
I, elena5100, understand that AI is ONLY to be used for tests, and th…
09c43a1 
…at every commit that I use AI for must start with 'AI Used'
@elena5100
implementation of BinarizingImageGroupFinder ( binarize image then f…
72374a3 
…ind groups)
@elena5100
AI Used... the tests not working
032b994
@xavierb117
AI Used - Temporary tests created by AI to help fix tests, will impro…
1b33722 
…ve later
@elena5100
fix it
bfe5be6
@elena5100
AI Used... still have issue with tests
3f21f35
@elena5100
Merge branch 'main' of https://github.com/xavierb117/centroid-finder
d7c1253
@xavierb117
Readded the deleted tests
c58edc7
timmyd4 and others added 30 commits November 10, 2025 23:32
@timmyd4
added test for thumbnail to check functionaility
ac88d1e
@xavierb117
Fixed bug with tracking files
348f7aa
@xavierb117
Merge branch 'server' of https://github.com/xavierb117/centroid-finder …
2dc2731 
…into server
@timmyd4
AI USED having trouble passing final tests after logic changes
7e72a09
@timmyd4
Created DockerPlan reademe file
6d4325f
@timmyd4
Added some of the plan
fa5f183
@timmyd4
Drafted DockerPlan
f0fcbb7
@timmyd4
added some more to the plan
5916315
@xavierb117
Added more content to the plan
04663f7
@xavierb117
AI Used - Used AI to help improve the server for Docker Preperation
58eac7e
@xavierb117
More Improvements made for Docker Preperation
b9c45c2
@xavierb117
Merge branch 'server' into image
54a79a6 
Merging Server improvements into Docker branch
@timmyd4
Merge branch 'server' into image
c9038e6
@timmyd4
Merge branch 'server' into image
f2a99f2
@xavierb117
Merge branch 'image' of https://github.com/xavierb117/centroid-finder …
0926bf5 
…into image
@timmyd4
AI USED, Created DockerFile
ddb07eb
@xavierb117
Merge branch 'image' of https://github.com/xavierb117/centroid-finder …
13035b1 
…into image
@timmyd4
AI USED, had to rearrange server code to work within the image on any…
b54a15a 
… device running our image
@timmyd4
did the same method for other logic on server code
c6af31f
@timmyd4
got routes working for image on docker
888c5d2
@xavierb117
Merge branch 'image' of https://github.com/xavierb117/centroid-finder …
8b8efa7 
…into image
@timmyd4
Readded process.cwd()
d4f22fa
@timmyd4
removed process.cwd()
567b57c
@timmyd4
added more time to process
1fc149f
@timmyd4
changed a var
92767f0
@timmyd4
changed the time to 1 min
d67c417
@timmyd4
changed from path.join to path.resolve
d14745a
@timmyd4
AI USED, changed dockerfile
cc407ad
@timmyd4
AI USED, added more code towards ffmpeg processing
fe4070c
@xavierb117
ENABLED CORS TO TEST IF FRONTEND CAN CATCH PICTURE - ONLY FOR FRONTEN…
356989b 
…D PROJECT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@xavierb117 @timmyd4 @elena5100