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.

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

Should cognoml reside in a separate repo? #64

Closed
dhimmel opened this issue Oct 23, 2016 · 5 comments
Closed

Should cognoml reside in a separate repo? #64

dhimmel opened this issue Oct 23, 2016 · 5 comments

Comments

@dhimmel
Copy link
Member

dhimmel commented Oct 23, 2016

cognoml is the Python package we're developing for machine-learning

I'm starting to think this would simplify things.

@awm33 what do you think?

@awm33
Copy link
Member

awm33 commented Oct 23, 2016

It might. So this repo would be the "playground", while cognoml is "production ready" and can be consumed by applications?

@dhimmel
Copy link
Member Author

dhimmel commented Oct 23, 2016

So this repo would be the "playground", while cognoml is "production ready" and can be consumed by applications?

Yeah, machine-learning will be a data analysis repository (tracks notebooks, figures, datasets). cognoml will be a python package repository. This will avoid confusion where it's not clear whether a file is needed for the package or for the exploratory analyses.

@awm33
Copy link
Member

awm33 commented Oct 23, 2016

That makes sense

@dhimmel
Copy link
Member Author

dhimmel commented Oct 24, 2016

Okay, I'll create a cognoml repo once #62 is merged.

From this StackOverflow answer, here's a beastly git command to take only the package-relevant files and rewrite history to include only relevant commits.

git filter-branch \
    --prune-empty \
    --index-filter '
        git ls-tree -z -r --name-only --full-tree $GIT_COMMIT \
        | grep --null-data --invert-match "^LICENSE.md$" \
        | grep --null-data --invert-match "^setup.py$" \
        | grep --null-data --invert-match "^cognoml" \
        | xargs --null --no-run-if-empty git rm --cached -r
    ' \
    -- \
    --all

This command currently will transfer the LICENSE.md and setup.py files to the new repo as well as everything in the cognoml directory.

@dhimmel
Copy link
Member Author

dhimmel commented Oct 25, 2016

Okay I used the above command to populate the new cognoml repository up to cognoma/cognoml@d4ab12c.

This was referenced Oct 25, 2016
dhimmel added a commit to dhimmel/machine-learning that referenced this issue Oct 25, 2016
The cognoml package now resides at https://github.com/cognoma/cognoml.

Closes cognoma#64.
This was referenced Oct 25, 2016
dhimmel added a commit that referenced this issue Nov 2, 2016
The cognoml package now resides at https://github.com/cognoma/cognoml.

Closes #64.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants