Move variables to reusable constant class #22

terrytangyuan · 2020-05-12T20:59:56Z

Variables/strings that appear in multiple places should be moved to a constant class so they can be easier to reuse, e.g.

the list of available metrics in core/scatter_plot.py and in metrics/bivariate.py. The doc-strings can point to this constant class directly.
Strings that represent file extensions in utilities/load_data.py.

* Add file extension changes * Make file type enum private * Fix unit test to be os independent * Fix nltk find (#155) * Fix issue link (#153) * Rename config_context to update_context (#154) * Add file extension changes * Make file type enum private * Fix unit test to be os independent * Fix import alias in tests

terrytangyuan added the Readability / Maintainability label May 12, 2020

terrytangyuan added this to To do in Open Source May 18, 2020

dvdjlaw self-assigned this Jul 7, 2020

dvdjlaw moved this from To do to In progress in Open Source Jul 8, 2020

terrytangyuan closed this as completed in eaabee6 Jul 15, 2020

Open Source automation moved this from In progress to Done Jul 15, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Move variables to reusable constant class #22

Move variables to reusable constant class #22

terrytangyuan commented May 12, 2020

Move variables to reusable constant class #22

Move variables to reusable constant class #22

Comments

terrytangyuan commented May 12, 2020