Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

[Documentation] How do I test MIC with different parameter values? #267

Open
dgarijo opened this issue Jul 24, 2020 · 4 comments
Open

[Documentation] How do I test MIC with different parameter values? #267

dgarijo opened this issue Jul 24, 2020 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@dgarijo
Copy link
Contributor

dgarijo commented Jul 24, 2020

As the documentation stands, it is unclear to me how to test MIC with different parameter values. We should state that you can do it by changing the default values in mic/mic.yaml

@dgarijo dgarijo added the documentation Improvements or additions to documentation label Jul 24, 2020
@dgarijo dgarijo self-assigned this Jul 24, 2020
@issue-label-bot
Copy link

Issue Label Bot is not confident enough to auto-label this issue. See dashboard for more details.

@mosoriob
Copy link
Contributor

I don't see the benefit.

@dgarijo
Copy link
Contributor Author

dgarijo commented Jul 24, 2020

When you are creating a component you want to test different parameter configurations to make sure it works.
Why you may want to do this?

When testing MIC, I have noticed that sometimes my parameters were hardcoded. By changing the defaults, I was able to verify that the right parameters were reflected in the execution. This required more than one execution.

@mosoriob
Copy link
Contributor

When testing MIC, I have noticed that sometimes my parameters were hardcoded. By changing the defaults, I was
able to verify that the right parameters were reflected in the execution. This required more than one execution.

That was a bug and we can create a unit test to avoid a future error, but we don't need to add a new option or documentation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants