Duane Hill wrote:
On Wed, 23 May 2007, Abba Communications wrote:
Since the introduction of SA v3.2.0, bayes_auto_learn_threshold_nonspam
appears to be -1.0.
from Duane
Duane and others....
With all sincere and due respect to the DEV's and their excellent hard
work...
I understand it takes a lot of work to maintain and appriciate the
efforts put forth.
Just how many places should this setting be???
Well, it version 3.1.8 it wasn't anywhere to be found in a configuration
file. You had to have the setting in place to deviate from 0.1.
So why not remove the setting from 10_default_prefs.cf (a core setting
file installed from the distribution) and have the code default do what
it should. Defaults are to handle issues where a setting isn't specified.
The code never changed. It still defaults to 0.1
Somewhere along the line, someone decided it should be -1. The only way
to change this is to:
1. Change the code and release a new version (stupid for such a small
change)
2. Make the change in a conf file by specifying a new value and
distribute that change to all using sa-update.
Obviously, the devs chose option 2. What this in effect does is change
the default autolearn threshold for the SA installation. It however
does not change the default in the code as sa-update cannot update core
SA code. It also cannot update the documentation that comes with SA.
I can understand it being in some type of conditional statement in
code if
the setting does not exist in the local.cf file, yet this is getting
to be a
tad confusing now...
It is somewhat confusing as if you were to read the documentation, it
says the default is 0.1. However, if you were to download SA and
install it without any modifications, the value that would be used for
this threshold would be -1. Being that devs can release conf changes
which can alter defaults, but they cannot update the documentation in
this manner, what do you suggest as an alternative? There really is no
other way to do it.
So, yes, its confusing, but there really isnt a better way to do it.
-Jim
