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

Rename "proc"/"pump" to "driver" and update format #23

Open
kjekac opened this issue Oct 23, 2024 · 0 comments
Open

Rename "proc"/"pump" to "driver" and update format #23

kjekac opened this issue Oct 23, 2024 · 0 comments
Assignees
@kjekac

Comments

@kjekac
Copy link
Contributor

kjekac commented Oct 23, 2024 •
edited
Loading

Procs should be called drivers. Additionally we should change their format from this

type Driver = Array (MsgID, Effect) -> MsgID -> Any -> Driver

to this

type Driver = MsgID -> Any -> (Driver, Array (MsgId, Effect))

Since this is more consistent with the format for cogs, where outputs aren't kept in the closure. The new docs (WIP; not yet published) already reflect this format change.
@kjekac kjekac self-assigned this Oct 23, 2024
@kjekac kjekac changed the title Change proc to driver and update format Rename "proc"/"pump" to "driver" and update format Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kjekac kjekac

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kjekac