seismometer icon indicating copy to clipboard operation
seismometer copied to clipboard
verified publisher icon epic-open-source

Contribute CohortsWidget

Open andli28 opened this issue 1 year ago • 1 comments

Overview

Description of changes

We have a ModelOptionsWidget, and a ModelOptionsAndCohortsWidget. This PR is to contribute the difference of the two widgets.

Author Checklist

  • [ ] Linting passes; run early with pre-commit hook.
  • [ ] Tests added for new code and issue being fixed.
  • [ ] Added type annotations and full numpy-style docstrings for new methods.
  • [ ] Draft your news fragment in new changelog/ISSUE.TYPE.rst files; see changelog/README.md.

andli28 avatar Aug 19 '24 18:08 andli28

Is there a reason why you are looking to add this particular widget?

It looks like you don't have any exploration controls that use the widget, which is confusing. It also looks like you add any unit tests for the new widget since you only changed one file.

gbowlin avatar Aug 21 '24 19:08 gbowlin

Included in #82

gbowlin avatar Oct 01 '24 21:10 gbowlin