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.

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

feat: allows functions to be combined and work on state #43

Conversation

younesStrittmatter
Copy link
Contributor

Description

Add functionality to combine functions and use the combined output on state

Copy link
Contributor

@benwandrew benwandrew left a comment

Choose a reason for hiding this comment

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

some comments and basic confusion on my part. please help set me straight!


# We can also pass parameters to the functions:
>>> def multiply(conditions, multiplier):
... return [el * multiplier for el in conditions]
Copy link
Contributor

Choose a reason for hiding this comment

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

i know it's arbitrary, but i assume 'el' is short for 'element'?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.

src/autora/state.py Outdated Show resolved Hide resolved
src/autora/state.py Outdated Show resolved Hide resolved
src/autora/state.py Outdated Show resolved Hide resolved
src/autora/state.py Show resolved Hide resolved
src/autora/state.py Show resolved Hide resolved
younesStrittmatter and others added 3 commits August 30, 2023 09:42
Copy link
Contributor

@benwandrew benwandrew left a comment

Choose a reason for hiding this comment

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

good stuff, and thanks for explaining it!

@younesStrittmatter younesStrittmatter added this pull request to the merge queue Aug 30, 2023
Merged via the queue into main with commit dddfa1e Aug 30, 2023
13 checks passed
@younesStrittmatter younesStrittmatter deleted the 42-add-some-basic-functionality-that-allows-multiple-experimentalists-to-be-combined-using-the-same-pool-of-conditions branch August 30, 2023 14:01
hollandjg added a commit that referenced this pull request Nov 29, 2023
feature!: allow other serializers and set built-in "pickle" as the default
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants