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

Update plasticc_pipeline #229

Open
Catarina-Alves opened this issue Mar 25, 2021 · 0 comments
Open

Update plasticc_pipeline #229

Catarina-Alves opened this issue Mar 25, 2021 · 0 comments
Assignees
@Catarina-Alves
Labels
enhancement Improvement to existing functionality or implementation, including adding a new functions/methods. low priority A "nice-to-have" low priority issue than can be tackled anytime. No rush for implementation

Comments

@Catarina-Alves
Copy link
Collaborator

The util plasticc_pipeline has many useful functions, such as create_folder_structure, get_directories, load_training_data. However, this file does not follow the code conventions and has some outdated names (e.g. load_training_data when this function can load any type of data).

I propose updating this file.

File: plasticc_pipeline.py
@Catarina-Alves Catarina-Alves added enhancement Improvement to existing functionality or implementation, including adding a new functions/methods. low priority A "nice-to-have" low priority issue than can be tackled anytime. No rush for implementation labels Mar 25, 2021
@Catarina-Alves Catarina-Alves self-assigned this Mar 25, 2021
@Catarina-Alves Catarina-Alves mentioned this issue Mar 25, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Catarina-Alves Catarina-Alves

Labels
enhancement Improvement to existing functionality or implementation, including adding a new functions/methods. low priority A "nice-to-have" low priority issue than can be tackled anytime. No rush for implementation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Catarina-Alves