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.

Sign up for GitHub

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

Jump to bottom

Split code into modules #2

Closed
gth828r opened this issue Feb 7, 2022 · 1 comment
Closed

Split code into modules #2

gth828r opened this issue Feb 7, 2022 · 1 comment
Assignees
@gth828r
Labels
enhancement New feature or request
Milestone
v0.1.0

Comments

@gth828r
Copy link
Owner

gth828r commented Feb 7, 2022

The code is all stuck in a single file right now. We should split it into modules.
@gth828r gth828r self-assigned this Feb 7, 2022
@gth828r gth828r added this to the v0.1.0 milestone Feb 7, 2022
@gth828r gth828r mentioned this issue Feb 7, 2022
Closed
gth828r added a commit that referenced this issue Feb 18, 2022
@gth828r
Convert brownify from a script to a package
d8539cd 
 * Split out the code into separate modules, leaving the brownify script
   in place
 * Add in module metadata (pyproject.toml, setup.cfg, setup.py) and
   remove requirements.txt
 * Update the versions for numpy and librosa, as the previous versions
   in did not work
 * Update gitignore to ignore build-related files

Addresses #2
@gth828r gth828r added the enhancement New feature or request label Feb 18, 2022
@gth828r
Copy link
Owner Author

gth828r commented Feb 18, 2022

I have split the code out into more maintainable modules and a script, and I also set up the infrastructure to do proper installs as a package. This one is all set now.

@gth828r gth828r closed this as completed Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gth828r gth828r

Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.1.0
Development

No branches or pull requests
1 participant
@gth828r