error 'BlenderVR_root' is not defined #32

jmespadero · 2015-04-08T10:23:07Z

I can't launch blenderVR after updating to current release.

I got an error in line 55 of blenderVR.py:
File "/home/jespa/bin/blender-vr/blender-vr/utils/blenderVR.py", line 55, in main
sys.path.append(os.path.join(BlenderVR_root, 'modules'))
NameError: name 'BlenderVR_root' is not defined

I have tried to define it in the previous line, just to receive the same error
in line 54 of console.py:
File "blender-vr/modules/blendervr/console/console.py", line 54, in init
self._update_loader_script = os.path.join(BlenderVR_root, 'utils',
NameError: name 'BlenderVR_root' is not defined

Defining the BLENDER_VR_ROOT enviroment variable does not help...

jmespadero · 2015-04-08T10:28:09Z

Ok, I think that the problem is the changes in your commit BlenderVR/blender-vr@14222ed

For example, in line 54 of console.py, you changed the variable name from blenderVR_root to BlenderVR_root

PyrApple · 2015-04-08T10:55:48Z

For now, I duplicated the current master to new branch: rename-BlenderVR
and reseted the master to commit prior to blenderVR change name.

we do need a dev branch :)

David

Le 8 avr. 2015 à 12:28, jmespadero notifications@github.com a écrit :

Ok, I think that the problem is the changes in your last commit

For example, in line 54 of console.py, you changed the variable name from blenderVR_root to BlenderVR_root

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

PyrApple · 2015-04-08T11:02:07Z

Apologies Dalai,

I backed up to

commit ff9d85d
Author: David Poirier-Quinot davipoir@gmail.com
Date: Tue Mar 31 19:01:02 2015 +0200

repaired mistake on win/linux configFiles

while everything was actually fine until the modifs you made @

commit 21bb8bb
Author: Dalai Felinto dfelinto@gmail.com
Date: Tue Apr 7 19:39:38 2015 -0300

Changes done during IEEE and soon after, mostly by David
Poirier-Quinot

* osc api documentation
* modified osc plugin, weird vehicle/user position issues in sent OSC msgs remain
* api-related comment modif
* my first contribution to blenderVR API, thanks Dash!
* modified .xml config
* added info in xml files
* added dk2-like .xml configuration file
* added requirement-dk2.txt for oculus rift dk2 install (pip3 six and websocket)
* Run Plugins for all machines (master and clients) -- it needs work
  still

(and can't find a quick way to fix for now)

anyway, everything untouched @ branch rename-BlenderVR.
Will look at it asap.

David

Le 8 avr. 2015 à 12:55, David Poirier-Quinot davipoir@gmail.com a écrit :

For now, I duplicated the current master to new branch: rename-BlenderVR
and reseted the master to commit prior to blenderVR change name.

we do need a dev branch :)

David

Le 8 avr. 2015 à 12:28, jmespadero notifications@github.com a écrit :

Ok, I think that the problem is the changes in your last commit

For example, in line 54 of console.py, you changed the variable name from blenderVR_root to BlenderVR_root

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

PyrApple · 2015-04-08T11:38:29Z

Fixed (thanks cherry-pick), last master commit is

commit a9971a3
Author: Dalai Felinto dfelinto@gmail.com
Date: Tue Apr 7 19:39:38 2015 -0300

Changes done during IEEE and soon after, mostly by David
Poirier-Quinot

* osc api documentation
* modified osc plugin, weird vehicle/user position issues in sent OSC msgs remain
* api-related comment modif
* my first contribution to blenderVR API, thanks Dash!
* modified .xml config
* added info in xml files
* added dk2-like .xml configuration file
* added requirement-dk2.txt for oculus rift dk2 install (pip3 six and websocket)
* Run Plugins for all machines (master and clients) -- it needs work
  still

remains BlenderVR rename and jmespadero's commits between
master and rename-BlenderVR

David

Le 8 avr. 2015 à 13:02, David Poirier-Quinot davipoir@gmail.com a écrit :

Apologies Dalai,

I backed up to

commit ff9d85d
Author: David Poirier-Quinot davipoir@gmail.com
Date: Tue Mar 31 19:01:02 2015 +0200
repaired mistake on win/linux configFiles
while everything was actually fine until the modifs you made @

commit 21bb8bb
Author: Dalai Felinto dfelinto@gmail.com
Date: Tue Apr 7 19:39:38 2015 -0300
Changes done during IEEE and soon after, mostly by David
Poirier-Quinot

* osc api documentation
* modified osc plugin, weird vehicle/user position issues in sent OSC msgs remain
* api-related comment modif
* my first contribution to blenderVR API, thanks Dash!
* modified .xml config
* added info in xml files
* added dk2-like .xml configuration file
* added requirement-dk2.txt for oculus rift dk2 install (pip3 six and websocket)
* Run Plugins for all machines (master and clients) -- it needs work
  still
(and can't find a quick way to fix for now)

anyway, everything untouched @ branch rename-BlenderVR.
Will look at it asap.

David

Le 8 avr. 2015 à 12:55, David Poirier-Quinot davipoir@gmail.com a écrit :

For now, I duplicated the current master to new branch: rename-BlenderVR
and reseted the master to commit prior to blenderVR change name.

we do need a dev branch :)

David

Le 8 avr. 2015 à 12:28, jmespadero notifications@github.com a écrit :

Ok, I think that the problem is the changes in your last commit

For example, in line 54 of console.py, you changed the variable name from blenderVR_root to BlenderVR_root

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

jmespadero · 2015-04-08T12:05:11Z

Well, almost...

I got in the log windows an error like this:
2015-04-08 14:03:11,803 No environments defined inside configuration file (line 1 of blender-vr/configurations/main-linux.xml)

To solve it, I had to edit the .xml file and change the first XML tag from blenderVR to BlenderVR

PyrApple · 2015-04-08T12:20:29Z

almost..

wen't back to pre-IEEE commits on master for now.

David

Le 8 avr. 2015 à 14:05, jmespadero notifications@github.com a écrit :

Well, almost...

I got in the log windows an error like this:
2015-04-08 14:03:11,803 No environments defined inside configuration file (line 1 of blender-vr/configurations/main-linux.xml)

To solve it, I had to edit the .xml file and change the tag to

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

dfelinto · 2015-04-08T12:26:50Z

The problem was a mistake in my end, I'm sorry. I didn't realize I had to rename non-Python files as well as the Python ones. Everything should be running smooth in master. (but yes, you need to rename your xml main tag to BlenderVR).

@PyrApple in cases like this you can simply: git revert HEAD (or git revert 7243dc1). git push --force should be avoided by all costs.

And yes we will have a 'dev' branch, but we are waiting for after the 1.0 tag is out. (yet another reason for me to double-check before pushing commits)

jmespadero · 2015-04-08T12:47:49Z

I'm still having problems... the daemons are launching but the keyboard and mouse didn't work anymore.
I'm not sure if it is my fault after all changes (because I'm quite new user with git). Tomorrow will try again from a new checkout.

Btw, don't feel bad, Dalai.

dfelinto · 2015-04-08T12:49:03Z

That's likely not related to the renaming, but I'll try one of the samples that depend on keyboard. Which setup are you trying? A single computer?

jmespadero · 2015-04-08T13:19:20Z

Yes, It is a single computer with the "Spider" demo and the "console" setting.

jmespadero · 2015-04-08T13:25:50Z

Ok, I have solved it doing a revert to a previous version.
Now, I'm using ce4a531

dfelinto · 2015-04-08T13:47:11Z

Here it's working fine BUT you need to use the latest sample files as well. There is also a chance you need to re-create the .spider.vr.blend file. So my suggestion is to get the latest samples and make sure you have no .*vr.blend around. Let me know how it goes

dfelinto closed this as completed Apr 8, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

error 'BlenderVR_root' is not defined #32

error 'BlenderVR_root' is not defined #32

jmespadero commented Apr 8, 2015

jmespadero commented Apr 8, 2015

PyrApple commented Apr 8, 2015

PyrApple commented Apr 8, 2015

PyrApple commented Apr 8, 2015

I backed up to

while everything was actually fine until the modifs you made @

jmespadero commented Apr 8, 2015

PyrApple commented Apr 8, 2015

dfelinto commented Apr 8, 2015

jmespadero commented Apr 8, 2015

dfelinto commented Apr 8, 2015

jmespadero commented Apr 8, 2015

jmespadero commented Apr 8, 2015

dfelinto commented Apr 8, 2015

error 'BlenderVR_root' is not defined #32

error 'BlenderVR_root' is not defined #32

Comments

jmespadero commented Apr 8, 2015

jmespadero commented Apr 8, 2015

PyrApple commented Apr 8, 2015

PyrApple commented Apr 8, 2015

I backed up to

while everything was actually fine until the modifs you made @

PyrApple commented Apr 8, 2015

I backed up to

while everything was actually fine until the modifs you made @

jmespadero commented Apr 8, 2015

PyrApple commented Apr 8, 2015

dfelinto commented Apr 8, 2015

jmespadero commented Apr 8, 2015

dfelinto commented Apr 8, 2015

jmespadero commented Apr 8, 2015

jmespadero commented Apr 8, 2015

dfelinto commented Apr 8, 2015