Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.

Use Python scripts to perform jobs that were being performed by C++ executables #1576

Closed
2 tasks done
david-ragazzi opened this issue Dec 1, 2014 · 7 comments
Closed
2 tasks done

Comments

@david-ragazzi
Copy link
Contributor

I'm working on setup.py in order to make it "pip-installable". The problematic point of this job is have to compile and install all current cpp executables and libraries. Some of these libraries (bindings) are inevitable to be build as extensions, however extensions like htmtest and compare_version can be replaced to pythonic versions. These 2 extensions are complicating the process as they should be installed on souce repository (bin folder) and not into nupic package folder like bindings. In the new version of setup.py that I mentioned only bindings are build.

When I create the PR solving #1573 , you will understand well what I'm trying explain.


@david-ragazzi david-ragazzi changed the title Leave "nupic" repository the more pythonic as possible Use Python scripts to perform jobs that were being performed by C++ executables Dec 2, 2014
@rhyolight rhyolight added this to the 0.1.0 milestone Dec 12, 2014
@rhyolight rhyolight modified the milestones: 0.2.0, 0.1.0 Dec 13, 2014
@david-ragazzi
Copy link
Contributor Author

@rhyolight Now this can be closed too because #1579

@rhyolight
Copy link
Member

@david-ragazzi I can't close this until the subtasks are complete. What about them?

@rhyolight rhyolight modified the milestones: 0.3.0, 0.2.0 Jan 21, 2015
@rhyolight rhyolight modified the milestones: 0.4.0, 0.3.0, 0.6.0 Feb 21, 2015
@david-ragazzi
Copy link
Contributor Author

All subtasks were complete. Closing....

@rhyolight
Copy link
Member

All subtasks were complete. Closing....

Eh? Neither subtask is actually closed in GitHub! numenta/nupic.core-legacy#257 & #1621

@rhyolight
Copy link
Member

Opening until we understand each other...

@rhyolight rhyolight reopened this Feb 26, 2015
@david-ragazzi
Copy link
Contributor Author

#1621 was solved with this:

https://github.com/numenta/nupic/blob/master/setup.py#L575

This issue was created because the code was in a c++ file to compare versions.. Look this:

https://github.com/numenta/nupic/pull/1579/files#diff-9ef085d0d82ff74621e7f677fdfa7ab9L1

@rhyolight
Copy link
Member

ok thanks

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

No branches or pull requests

2 participants