drivendataorg / cookiecutter-data-science

A logical, reasonably standardized, but flexible project structure for doing and sharing data science work.
https://cookiecutter-data-science.drivendata.org/
MIT License
7.99k stars 2.41k forks source link

Remove config import from init to fix scaffold usage #372

Closed pjbull closed 1 month ago

pjbull commented 1 month ago

Closes #370

jayqi commented 1 month ago

not needed

The main thing that happens in config right now that is valuable to auto-execute is this bit:

https://github.com/drivendataorg/cookiecutter-data-science/blob/bbfa1511ff724731ddd482bcffcd62fb1e6b1b83/%7B%7B%20cookiecutter.repo_name%20%7D%7D/%7B%7B%20cookiecutter.module_name%20%7D%7D/config.py#L24-L32

pjbull commented 1 month ago

The main thing that happens in config right now that is valuable to auto-execute is this bit:

Ah, that's a good point. Added back in.

pjbull commented 1 month ago

Just summarizing for my own understanding / posterity:

Nailed it! 🔨