Async runtime #5

Gavin-Niederman · 2023-10-16T21:28:15Z

Add a basic async runtime to pros-rs and make Robot an async trait.

todo before merging:

Test the async runtime on real vex hardware

add: NEXT_LOCAL_INDEX constant

doinkythederp

Some thoughts and questions I had

pros/examples/accessories.rs

pros/src/async_runtime/executor.rs

pros/src/async_runtime/mod.rs

pros/src/motor.rs

Gavin-Niederman

remove unused marker

pros/src/async_runtime/mod.rs

refactor(async): simplify runtime using async-task

fix: remove uneccesarry mutable static

doinkythederp

Just some things that might be mistakes and then let's merge

pros/examples/accessories.rs

pros/examples/basic.rs

pros/src/async_runtime/executor.rs

pros/src/async_runtime/mod.rs

doinkythederp

LGTM (Lets Go To vex and ask them for Money) 🚀

Gavin-Niederman and others added 26 commits October 2, 2023 19:14

add: async runtime implementation

61a3900

fix: switch to concurrent_queue queues

4d8ddf4

fix: add new function to Executor

c3711d6

fix: switch to unsafecell instad of mutex

5677729

update todo list

d730f1a

start implementing final executor design

d475594

fix: switch to storing futures in raw pointers

1113e4c

fix: task internal now can poll futures

b622dad

feat: executor tick and run functions

7b4be88

feat: block_on function

840a144

add: ready macro

7503003

feat: task_local! macro

c33ddc9

feat: new executor

ac725ae

fix: add block_on function

40b5f95

fix: block on requires future to be static

9b8d938

add: NEXT_LOCAL_INDEX constant

Merge branch 'main' into async_runtime

a3a5ddb

feat: make Robot trait functions async

d4e94ab

add: simple motor stopped future

4affa3d

chore: clippy

430ee75

add: wakers and async sleep function

9450fb6

fix: initialize main thread tls

bff860e

fix: call init main

fa7a840

fix: change task local storage method

af43133

add: reactor

b2fe5bd

fix: drive all futures to completion

fdccd4c

feat: future join handles

e5ec7a6

Gavin-Niederman added the type:enhancement New feature or request label Oct 16, 2023

Gavin-Niederman requested a review from doinkythederp October 16, 2023 21:28

Gavin-Niederman assigned doinkythederp and Gavin-Niederman Oct 16, 2023

doinkythederp suggested changes Oct 16, 2023

View reviewed changes

Gavin-Niederman commented Oct 17, 2023

View reviewed changes

pros/src/async_runtime/mod.rs Outdated Show resolved Hide resolved

Gavin-Niederman and others added 22 commits October 16, 2023 19:18

docs: document async executor functions

6c85e72

fix: remove unused marker field

3635741

docs(async_runtime): specify the benefits of spawn()

8d9b65a

fix: correct slightly incorrect docs on spawn

39c0fa4

feat: support panic messages in simulator

3b73a5e

fix: panic messages

6fd1957

add: basic example

b686f34

feat: complete_timeout

6ca9686

refactor: rename task_local to os_task_local

046da3c

refactor(async): simplify runtime using async-task

6b8ed48

fix(async): only poll task when woken

2ab4ce2

fix: use CString directly

ef4c69c

fix: remove std dependency from task init code

e4a1c01

Merge branch 'async_runtime' into async-simplified

7b1a86c

Merge pull request #6 from max-niederman/async-simplified

16c290e

refactor(async): simplify runtime using async-task

Merge branch 'main' into async_runtime

e589833

feat: seperate async and sync robot traits

54ee3d2

fix: get rid lockup in task locals

72348f1

refactor: change task module structure

5698cdf

fix: remove uneccesarry mutable static

Merge branch 'main' into async_runtime

d556ce8

update todo list

731533b

fix: allow async task spawning in an async task

5dd3021

doinkythederp suggested changes Dec 22, 2023

View reviewed changes

pros/examples/accessories.rs Outdated Show resolved Hide resolved

pros/examples/basic.rs Outdated Show resolved Hide resolved

pros/src/async_runtime/executor.rs Show resolved Hide resolved

pros/src/async_runtime/mod.rs Show resolved Hide resolved

Gavin-Niederman added 3 commits December 22, 2023 14:54

fix(examples): sleeps in accessories and typo in basic

f51fbdd

add: async_runtimecomplete_all function

5b34c62

chore: formatting

591fa84

doinkythederp approved these changes Dec 22, 2023

View reviewed changes

Gavin-Niederman merged commit 23c3454 into main Dec 22, 2023
6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Async runtime #5

Async runtime #5

Gavin-Niederman commented Oct 16, 2023 •

edited by doinkythederp

Loading

doinkythederp left a comment

Gavin-Niederman left a comment

doinkythederp left a comment

doinkythederp left a comment

Async runtime #5

Async runtime #5

Conversation

Gavin-Niederman commented Oct 16, 2023 • edited by doinkythederp Loading

doinkythederp left a comment

Choose a reason for hiding this comment

Gavin-Niederman left a comment

Choose a reason for hiding this comment

doinkythederp left a comment

Choose a reason for hiding this comment

doinkythederp left a comment

Choose a reason for hiding this comment

Gavin-Niederman commented Oct 16, 2023 •

edited by doinkythederp

Loading