search

lejon / PartiallyCollapsedLDA

Implementations of various fast parallelized samplers for LDA, including Partially Collapsed LDA, Light LDA, Partially Collapsed Light LDA and a very efficient Polya-Urn LDA
26 stars 20 forks source link

base_output_dir usage / bug #20

Open rbbby opened 2 years ago

rbbby commented 2 years ago

I am not able to get the arguments _base_outdir and _experiment_outdir to work after trying out many combinations and exploring the source code. It would be great to add some example in the configuration docs or to check if there is some bug involved.

Currently I am working around the problem by changing the default value set in LDAConfiguration.java as:

public static final String BASE_OUTPUT_DIR_DEFAULT = base/out/dir

This works after rebuilding the module. But specifying it as a model config argument would be much more convenient.

lejon commented 1 year ago

This was indeed a bug. Should hopefully be fixed in 9.2.1 (worked for me :) )