Skip to content
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

redo of integrating cluster estimation worker into main #209

Closed
wants to merge 46 commits into from
Closed

redo of integrating cluster estimation worker into main #209

wants to merge 46 commits into from

Conversation

Aleksa-M
Copy link
Contributor

@Aleksa-M Aleksa-M commented Oct 9, 2024

No description provided.

Cyuber added 30 commits September 26, 2024 11:17
@Cyuber
implemented string field for ObjectInWorld class to specify which typ…
c517762 
…e of real world object it is.
@Cyuber
fixed mistakenly changed annotation in ClusterElimination.run
3aad8c7
@Cyuber
fixed linters
2dfe21b
@Cyuber
fixed black --check issues
b9043b0
@Cyuber
fixed pylint issues
2577104
@Cyuber
fixed linters in test_landing_pad_tracking.py
209d372
@Cyuber
fixed linter issues in test_decision.py
c40ddbd
@Cyuber
fixed black --check issue with test_decision.py
99140d9
@Cyuber
fixed test_cluster_detection failed test case
4b1424d
@Cyuber
changed object_type: string to label: int, also made it keyword param…
676bcd1 
…eter with default 0
@Cyuber
went back to old version of main
647f73a
@Cyuber
initial commit for branch, work in progress
360a6ce
@Cyuber
implemented string field for ObjectInWorld class to specify which typ…
eff5831 
…e of real world object it is.
@Cyuber
fixed mistakenly changed annotation in ClusterElimination.run
451e0e5
@Cyuber
fixed linters
c4f3cb9
@Cyuber
fixed black --check issues
bc2c6f6
@Cyuber
fixed pylint issues
cf55400
@Cyuber
fixed linters in test_landing_pad_tracking.py
7dd1e9f
@Cyuber
fixed linter issues in test_decision.py
a5da749
@Cyuber
fixed black --check issue with test_decision.py
96618fb
@Cyuber
fixed test_cluster_detection failed test case
b3f068d
@Cyuber
changed object_type: string to label: int, also made it keyword param…
0916b10 
…eter with default 0
@Cyuber
initial commit for branch, work in progress
652e1da
@Cyuber
used black --preview to reformat the code
7476ddb
@Cyuber
Merge branch 'integrating-cluster-estimation-worker' of https://githu…
adddd92 
…b.com/UWARG/computer-vision-python into integrating-cluster-estimation-worker
@Cyuber
ran black . to reformat
bc65465
@Cyuber
changed label annotation
4b6abb6
@Cyuber
syntax error
808c4a2
@Cyuber
added values back for label
a58c56a
@Cyuber
fixed errors that formatter made
c74a1f4
mgupta27
mgupta27 reviewed Oct 9, 2024
View reviewed changes
Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

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

Reviewed.

config.yaml Outdated Show resolved Hide resolved
config.yaml Outdated Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
main_2024.py Show resolved Hide resolved
mgupta27
mgupta27 reviewed Oct 12, 2024
View reviewed changes
Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

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

Reviewed. Linters are failing. Please run black --check . to see if the code looks all good and black . to fix any files that cause an issue.

main_2024.py Outdated Show resolved Hide resolved
main_2024.py Outdated Show resolved Hide resolved
main_2024.py Outdated Show resolved Hide resolved
mgupta27
mgupta27 reviewed Nov 1, 2024
View reviewed changes
Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

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

Reviewed.

config.yaml Outdated Show resolved Hide resolved
main_2024.py Outdated Show resolved Hide resolved
mgupta27
mgupta27 approved these changes Nov 2, 2024
View reviewed changes
Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

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

Approved!

Aleksa-M and others added 10 commits November 7, 2024 01:50
@Aleksa-M
redo of integrating cluster estimation worker into main
617ae90
@Aleksa-M
redo of integrating cluster estimation worker into main
05fdf63
@Cyuber @Aleksa-M
implemented changes from review
d86f7ce
@Aleksa-M
fixed formatting issue
a203f42
@Aleksa-M
implemented reviewed changes
4e0602f
@Aleksa-M
fixed review changes
c84fe86
@Aleksa-M
rebased
f0aaaa9
@Aleksa-M
Merge branch 'cluster-estimation-worker-integration-redo' of https://…
db684fa 
…github.com/UWARG/computer-vision-python into cluster-estimation-worker-integration-redo
@Aleksa-M
Merge branch 'cluster-estimation-worker-integration-redo' of https://…
3f799e0 
…github.com/UWARG/computer-vision-python into cluster-estimation-worker-integration-redo
@Aleksa-M
Merge branch 'cluster-estimation-worker-integration-redo' of https://…
5001a51 
…github.com/UWARG/computer-vision-python into cluster-estimation-worker-integration-redo
@maxlou05 maxlou05 closed this Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgupta27 mgupta27 mgupta27 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Aleksa-M @mgupta27 @maxlou05 @Cyuber