Removed some configuration options form dynamic list.
authorYeGoblynQueenne@splinter <ep50@uni.brighton.ac.uk>
Mon, 29 Aug 2016 08:36:43 +0000 (11:36 +0300)
committerYeGoblynQueenne@splinter <ep50@uni.brighton.ac.uk>
Mon, 29 Aug 2016 08:36:43 +0000 (11:36 +0300)
* evaluation_parsing_time_limit/1 is only relevant to testing, so let's
  leave it aside for the moment until I have some sort of environment
  variable that tells us when specific options are relevant (and
  therefore should be reported to the user) or not.
* output_type/1 I have never edited dynamically, I believe. I guess it
  would be nice to have and all but one point of the dynamic
  configuration predicates is to make it easier to figure out what is
  the minimum set of configuration options needed to run the program, so
  it's a good idea to unclutter often.

lib/project_utilities.pl
tree_learning/configuration.pl

index b1f02b3..680873c 100644 (file)
@@ -746,10 +746,8 @@ dynamic_configuration(Stream):-
        configuration_options(Os)
        ,findall(O-Vs
                ,(member(O-Vs,Os)
-                % @TODO: this is now relevant to all testing.
                 % Only show gnf-related options when they're relevant.
                 ,(   (    O == production_arity
-                     ;    O == evaluation_parsing_time_limit
                      )
                  ->  (   configuration:transformation_format(gnf)
                      ;   configuration:transformation_format(stochastic_gnf)
index 4254ca1..37e72c1 100644 (file)
@@ -35,9 +35,7 @@
          ,graph_arity/1
          ,language_file_name/1
          ,lexicalisation_strategy/1
-         ,output_type/1
          ,production_arity/3
-         ,evaluation_parsing_time_limit/1
          ,training_set_size/1
          ,transformation_format/1
          ].
@@ -552,7 +550,7 @@ testing_protocol(strings).
 %      held-out test set will be the complement of this number to 1,
 %      duh.
 %
-training_set_size(0.9).
+training_set_size(1.0).
 
 
 %!     transformation_format(?Format) is det.