Setting DKPRO_HOME in demos by default

google-code-export / dkpro-tc

Automatically exported from code.google.com/p/dkpro-tc

Other

1 stars 0 forks source link

Setting DKPRO_HOME in demos by default #151

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

When running the TwentyNewsgroupsDemo as is, I get such an exception:

---

Exception in thread "main" 
org.springframework.beans.factory.BeanDefinitionStoreException: Invalid bean 
definition with name 'StorageService' defined in class path resource 
[META-INF/spring/context.xml]: Could not resolve placeholder 'DKPRO_HOME' in 
string value [${DKPRO_HOME}/de.tudarmstadt.ukp.dkpro.lab/repository]

---

How about to explicitly set the DKPRO_HOME in the example code so that the 
examples run out of the box without modifying the run configuration?

System.setProperty("DKPRO_HOME", "target/TwentyNewsgroupsDemo")

Original issue reported on code.google.com by richard.eckart on 16 Jun 2014 at 9:55

GoogleCodeExporter commented 9 years ago

The tutorial clearly shows that/how DKPRO_HOME needs to be set: 
https://code.google.com/p/dkpro-tc/wiki/DemoExperiments.

However, I agree, we should consider adding this line to demo examples by 
default, since users who didn't read through the tutorial might be discouraged 
by that error message.

Original comment by daxenber...@gmail.com on 17 Jun 2014 at 7:40

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

I agree, we should set that.
I had some students also running into that problem, so it seems (as usual) you 
should not assume that documentation is actually read ;)

Original comment by torsten....@gmail.com on 17 Jun 2014 at 7:42

GoogleCodeExporter commented 9 years ago

I'll also check if I can convince the lab to use a default value for DKPRO_HOME 
if it is not set. Won't promise this in 0.11.0 though.

Original comment by richard.eckart on 17 Jun 2014 at 7:57

GoogleCodeExporter commented 9 years ago

Original comment by daxenber...@gmail.com on 4 Jul 2014 at 9:36

Added labels: Milestone-Release0.7.0

GoogleCodeExporter commented 9 years ago

Original comment by daxenber...@gmail.com on 29 Jul 2014 at 10:00

Changed title: Setting DKPRO_HOME in demos by default

GoogleCodeExporter commented 9 years ago

This issue was closed by revision r1053.

Original comment by torsten....@gmail.com on 3 Sep 2014 at 8:50

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Original comment by daxenber...@gmail.com on 3 Sep 2014 at 9:05