Remove stream config

[ocefpaf]

Start to address #99

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[ocefpaf]

@kthyng sorry to ping you here but who would be the best person to review this one?

[kthyng]

@ocefpaf No problem! We're discussing internally about getting some people involved.

[iwensu0313]

Hi @ocefpaf - thanks for taking a stab here. @melaniefeen and I took a look at it together. These changes to documentation look good.

We did notice that StreamConfig is still referenced in the following places:

• ContextConfig section of the Usage page
• Comments and docstrings in config.py e.g. ContextConfig class definition docstring

But StreamConfig objects don't seem to exist anymore in the most recent version of the ioos_qc package. Digging around to figure out what the replacement is. Perhaps just Config? Any ideas?

[iwensu0313]

Should StreamConfig be replaced w/ something else here? What's the new object called?

[ocefpaf]

ContextConfig can take any ConfigType if I understood the code correctly. I changed to that instead of StreamConfig.

[iwensu0313]

Yea I see that now, ConfigType object is what we're passing when initializing ContextConfig class.

[ocefpaf]

But StreamConfig objects don't seem to exist anymore in the most recent version of the ioos_qc package. Digging around to figure out what the replacement is. Perhaps just Config? Any ideas?

Quoting Kyle from #99 (comment):

The docs need to be updated, the only interface you need to load any config object is the Config class: from ioos_qc.config import Config

My guess is that this is already documented, right? So it is a matter of just removing StreamConfig. I'll remove it from the places you pointed out.

[iwensu0313]

Looks good to me! @ocefpaf