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

Migrate task planning #14

Open
wants to merge 106 commits into
base: main
Choose a base branch
from
Open

Migrate task planning #14

wants to merge 106 commits into from

Conversation

JaredBaileyDuke
Copy link

@JaredBaileyDuke JaredBaileyDuke commented Oct 26, 2024 •
edited by maxwellmlin
Loading

Draft PR for task_planning migration. Do not merge until task_planning has been tested.

maxwellmlin and others added 30 commits September 28, 2024 16:46
@maxwellmlin
Copy over ROS files
2dc09cf
@maxwellmlin
Add entry point
3931921
@maxwellmlin
Port task_runner
d28faa6
@maxwellmlin
Pass self to interfaces
da47caa
@maxwellmlin
hi
2997da7
@maxwellmlin
Add todo comment
21a1fb9
@JaredBaileyDuke
part way through CV update to ROS 2
8904c15
@JaredBaileyDuke
updated __init__ of Class in its entirety to ROS 2
d1c8d98
@JaredBaileyDuke
Added all updates to ROS 2
86345e8
@maxwellmlin
Merge branch 'main' into migrate-task-planning
a171269
@vedarshshah
Merge branch 'main' into migrate-task-planning
bbab34a
@JaredBaileyDuke
Updated cv.py file to match OOP standards, removed comments, created …
e78388f 
…constant QUEUE_SIZE
@drago314
Updated all interfaces to rclpy
714a541
@drago314
Fix Imports
facdee3
@drago314
Fix Some Imports
413ec80
@SatvikMatta
Added much needed and overdue comments to task_runner.py.
35300f1
@maxwellmlin
Add ros-jazzy-robot-localization to dockerfile
e0b0ebf
@maxwellmlin
Add python3-jsonpickle to Dockerfile
a98051f
@maxwellmlin
Merge remote-tracking branch 'refs/remotes/origin/migrate-task-planni…
0d5ff68 
…ng' into migrate-task-planning
@SatvikMatta
WE SOLVED YOUR CONTROLS, CV, AND STATE PROBLEMS. YOU'RE WELCOMEros2 r…
feaadf1 
…un task_planning task_runnerros2 run task_planning task_runnerros2 run task_planning task_runnerros2 run task_planning task_runner! BOOMros2 run task_planning task_runnerros2 run task_planning task_runner!
@maxwellmlin
Add .flake8 rules
b6eac51
@maxwellmlin
Ignore D100
70a9d84
@maxwellmlin
Clean up task_runner
c45e812
@maxwellmlin
Add ros2 TODO suffix
52cfab3
@SatvikMatta
Fix Small issues
20ac743
@SatvikMatta
Small Changes
a99de32
@SatvikMatta
Small Change
90900f3
@maxwellmlin
Fix task_runner
26f7767
@maxwellmlin
Merge remote-tracking branch 'refs/remotes/origin/migrate-task-planni…
79c6534 
…ng' into migrate-task-planning
@maxwellmlin
Fix keyword order
efe9a14
maxwellmlin and others added 25 commits October 26, 2024 23:44
@maxwellmlin
revert
f2aee85
@maxwellmlin
Add TODO comments
d9fc921
@vedarshshah
Added full support for actions in jsonpickle custom handlers
6e52f04
@maxwellmlin
Merge remote-tracking branch 'refs/remotes/origin/migrate-task-planni…
0ea6a67 
…ng' into migrate-task-planning
@maxwellmlin
Use @singleton for TaskUpdatePublisher
499b4d0
@maxwellmlin
Bypass odom and state waits
b9f916c
@drago314
Attempt To Fix Launch File
10f3472
@drago314
Fix task_runner xml launch file
1da5c44
@drago314
Fix some timer issues
97eabb2
@drago314
Fix Runtime Error in self._enable_controls(enable)
447d03d
@drago314
Successful test of two sequential tests
a879ab9
@drago314
Successful Test of Task Calling Task that Yields
571f3d8
@drago314
Fix Potential Bug
cc0cd96
@drago314
Fix Some Todo:Ros2
8505b5c
@maxwellmlin
Resolve TODO:ros2 and fix minors bugs
d3546fc
@maxwellmlin
Set default ros args
d21db4a
@maxwellmlin
Fix header timestamp bug
f97cfb6
@maxwellmlin
Some more testing
ca04d3d
@maxwellmlin
Update README
504dfc7
@maxwellmlin
Update README
3408a6e
@maxwellmlin
Update metadata
ca5b4b6
@maxwellmlin
Review test_tasks
917ba01
@maxwellmlin
fix: kill timer once tasks done
f62043f
@maxwellmlin
Adjust message
1f2e3d8
@maxwellmlin
Merge branch 'main' into migrate-task-planning
1db0d50
@maxwellmlin maxwellmlin marked this pull request as ready for review November 9, 2024 16:36
@maxwellmlin
Copy link
Member

Task planning package ready for review. Still need to lint and test (see #3).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxwellmlin maxwellmlin Awaiting requested review from maxwellmlin

@vedarshshah vedarshshah Awaiting requested review from vedarshshah

At least 1 approving review is required to merge this pull request.

Assignees

@SatvikMatta SatvikMatta

@drago314 drago314

@JaredBaileyDuke JaredBaileyDuke

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Task Planning Migration
5 participants
@JaredBaileyDuke @maxwellmlin @SatvikMatta @drago314 @vedarshshah