error 'BlenderVR_root' is not defined about source

[jmespadero]

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]

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 [email protected] 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]

Apologies Dalai,

I backed up to

commit ff9d85d
Author: David Poirier-Quinot [email protected]
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 [email protected]
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 [email protected] 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 [email protected] 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]

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

---

commit a9971a3
Author: Dalai Felinto [email protected]
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 [email protected] a écrit :

Apologies Dalai,

I backed up to

commit ff9d85d
Author: David Poirier-Quinot [email protected]
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 [email protected]
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 [email protected] 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 [email protected] 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]

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]

almost..

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

David

Le 8 avr. 2015 à 14:05, jmespadero [email protected] 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]

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]

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]

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]

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

[jmespadero]

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

[dfelinto]

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