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

Add Butterfinger, change the API #3

Merged
merged 2 commits into from
May 31, 2024
Merged

Add Butterfinger, change the API #3

merged 2 commits into from
May 31, 2024

Conversation

philipp-jung
Copy link
Collaborator

This was intended to add the Butterfinger error type, which it does. It also completely changes how ErrorMechanism and ErrorType is configured.

The idea is that the abstract base classes ErrorMechanism and ErrorType become stateful, and that they are configured upon initialization.

I made this change because I figured that ErrorTypes need to be configured with configurations that are very different. Having a dataclass that stores that configuration allows us to control all parameters in one place, which is handy when adding more ErrorTypes in the future I believe.

This does change however how error_generation can be used, check the samples.ipynb for reference.

This was intended to add the Butterfinger error type, which it does. It
also completely changes how ErrorMechanism and ErrorType is configured.

The idea is that the abstract base classes ErrorMechanism and ErrorType
become stateful, and that they are configured upon initialization.

I made this change because I figured that ErrorTypes need to be
configured with configurations that are very different. Having a
dataclass that stores that configuration allows us to control all
parameters in one place, which is handy when adding more ErrorTypes in
the future I believe.

This does change however how error_generation can be used, check the
samples.ipynb for reference.
Make it so that the ErrorMechanism Objects take up to two arguments when
being created, and not a config-Object, which is a bit weird.
@philipp-jung philipp-jung merged commit e6acb9f into main May 31, 2024
18 checks passed
@philipp-jung philipp-jung deleted the feat/butterfinger branch May 31, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant