You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now, sparsecoding/data contains dictionary loading code, dictionary files, and transform code. This makes it less intuitive to use as code to transform data and load specific types of data are under the same umbrella.
I propose consolidating code such that there are sparsecoding/data/dictionaries.py which manages the loading of dictionaries (what is currently in sparsecoding/data/utils.py which can also use documentation), and sparsecoding/data/datasets.py which manages the load of datasets (contains the code in sparsecoding/data/datasets/bars.py and sparsecoding/data/datasets/field.py.
A sparsecoding/transforms directory can then be made that contains the all of the transform .pys. Additionally
This should probably be done after @HarrisonSantiago merges his contributions to the whitening code. Also, we'll have to update the example notebooks to reflect these new changes.
The text was updated successfully, but these errors were encountered:
Right now,
sparsecoding/data
contains dictionary loading code, dictionary files, and transform code. This makes it less intuitive to use as code to transform data and load specific types of data are under the same umbrella.I propose consolidating code such that there are
sparsecoding/data/dictionaries.py
which manages the loading of dictionaries (what is currently insparsecoding/data/utils.py
which can also use documentation), andsparsecoding/data/datasets.py
which manages the load of datasets (contains the code insparsecoding/data/datasets/bars.py
andsparsecoding/data/datasets/field.py
.A
sparsecoding/transforms
directory can then be made that contains the all of the transform.py
s. AdditionallyThis should probably be done after @HarrisonSantiago merges his contributions to the whitening code. Also, we'll have to update the example notebooks to reflect these new changes.
The text was updated successfully, but these errors were encountered: