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

Moved all prox and solver source code #140

Closed
wants to merge 8 commits into from

Conversation

stephanegaiffas
Copy link
Collaborator

Second iteration for the reorganization and flattening of tick, see #124

  • Moved all prox and solver python modules and C++ source code
  • Updated config files for compilation with mkn
  • Updated and reorganized documentation

@@ -1 +0,0 @@
# License: BSD 3 clause
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file must be kept for pip distribution (and moved to tick/prox/build/__init__.py)

@@ -1 +0,0 @@
# License: BSD 3 clause
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file must be kept for pip distribution (and moved to tick/solver/build/__init__.py)

@Mbompr
Copy link
Contributor

Mbompr commented Dec 1, 2017

Without these files I get
ImportError: No module named 'tick.prox.build.prox' error as well.

I think you have these files on your computer but you need to tell git to track them manually as they are ignored. Either by typing git add -f tick/prox/build/__init__.py either by tweaking .gitignore file, this would be cleaner I don't know how to set a general rule for all __init__.py under build folders.

@stephanegaiffas stephanegaiffas force-pushed the grand-flattening branch 2 times, most recently from e29c03d to 56ae7ea Compare December 4, 2017 09:50
@PhilipDeegan
Copy link
Member

Is this superseded by #143 ?

@stephanegaiffas
Copy link
Collaborator Author

This is superseded by #145

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

Successfully merging this pull request may close these issues.

None yet

3 participants