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

Fuzzer should have init_state() method #2431

Open
tokatoka opened this issue Jul 22, 2024 · 4 comments
Open

Fuzzer should have init_state() method #2431

tokatoka opened this issue Jul 22, 2024 · 4 comments

Comments

@tokatoka
Copy link
Member

todo for after generic refactor

fuzzer should have a simple init_state() method that calls feedback.init_state and objectives.init_state()
currently we do it in side state::new(), but this feels odd

@domenukk
Copy link
Member

Why is it odd to initialize the state in State::new()? Feels somewhat correct to me

@tokatoka
Copy link
Member Author

state::new() is a constructor for the StdState. it has nothing to do with the metadata initialization

@domenukk
Copy link
Member

Shouldn't the state be returned fully initialized?
Or do you want to move the init method to the State trait?

@tokatoka
Copy link
Member Author

Or do you want to move the init method to the State trait?

yes this is more correct.

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

No branches or pull requests

2 participants