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

Create config yaml file(s) for use by tests #137

Closed
jamessynge opened this issue Nov 26, 2017 · 2 comments · Fixed by #951
Closed

Create config yaml file(s) for use by tests #137

jamessynge opened this issue Nov 26, 2017 · 2 comments · Fixed by #951
Labels
config Items related to configuration or the config file. Documentation
Milestone

Comments

@jamessynge
Copy link
Contributor

I want a config file that can be used by the dome tests, and it makes sense to have another such config file for the bisque tests.

The existing POCS/conf_files/pocs.yaml seems to really be the local file for PAN001. Perhaps we need to have some other file as the base yaml file?

Regarding tests, we could use a separate file such as pocs_test.yaml, or even multiple such files so that we can test the effect of having different numbers of cameras, or different state machines. I can't add a dome section to pocs.yaml without it affecting PAN001 and PAN006.

@wtgee
Copy link
Member

wtgee commented Nov 26, 2017

Sorry, this is mostly a documentation issue.

$POCS/conf_files/pocs.yaml is the default config file that should be used for tests and to verify the software runs. It's mostly meant as an example. Any file in the $POCS/conf_file/ directory can have a foo_local.yaml version that will override and add to the foo.yaml file. pocs.yaml should never be altered (for normal usage), which is why it is part of the repository. A local unit should therefore have a $POCS/conf_files/pocs_local.yaml file for any local-specific settings.

@wtgee wtgee mentioned this issue Aug 25, 2019
@wtgee wtgee mentioned this issue Feb 16, 2020
2 tasks
@wtgee
Copy link
Member

wtgee commented Feb 16, 2020

Will be closed by #951.

@wtgee wtgee added the config Items related to configuration or the config file. label Feb 16, 2020
@wtgee wtgee modified the milestones: V1.0, v0.8.0 Feb 17, 2020
@wtgee wtgee closed this as completed in #951 Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config Items related to configuration or the config file. Documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants