Remove $NUPIC dependency

[passiweinberger]

had an issue with the NUPIC variable, heared you were about to remove them anyways, so I just did it :) Hope this helps :)

fixes #1947

[rhyolight]

Thanks, @passiweinberger. I will try to test this out Monday.

[rhyolight]

Although this fixes tests, not sure it fixes the general swarming problem.

[passiweinberger]

Yes. True, I'll get my hand on it 2day. Sorry don't merge yet

-------- Original message --------

From: Matthew Taylor notifications@github.com

Date:03/30/2015 03:48 (GMT+01:00)

To: numenta/nupic nupic@noreply.github.com

Cc: Pascal Weinberger passiweinberger@gmail.com

Subject: Re: [nupic] Remove $NUPIC dependency (#1968)

Although this fixes tests, not sure it fixes the general swarming problem.

—
Reply to this email directly or view it on GitHub.

[passiweinberger]

But wanted to make sure this way to fix it is acceptable :)

[pehlert]

I had a go at removing all $NUPIC dependencies, too. Saw too late, that @passiweinberger was working on it. It's surely a more controversial approach and I'm not sure if that's in line with Python best practices, but please have a look: #1970

[passiweinberger]

@pehlert s fix #1970 works. Close this one, no need for dublicates :D

[passiweinberger]

Now it removes the NUPIC path completely as suggested by @david-ragazzi in #1970 if we decide to do it :)

[david-ragazzi]

HI @passiweinberger , I think it's better we keep a single PR for this issue. :-/ So how about push this change and adapt to my suggestion (https://github.com/numenta/nupic/pull/1970/files#r27405407) into @pehlert branch. This way we centralize efforts.. :-)

[passiweinberger]

(Y)
How do I do this btw. ? :P

[rhyolight]

Seems to me like #1970 should be merged first, and then this PR should be updated to use NUPIC_ROOT?

[rhyolight]

blocked by #1970

[passiweinberger]

#2205