Commit Graph

4828 Commits

Author SHA1 Message Date
Al
1b2696b3b5 [utils] adding string_is_digit function, similar to Python\'s (i.e. counts if it's in the Nd unicode category) 2017-03-15 13:04:39 -04:00
Al
1a1f0a44d2 [parser] parser only inserts spaces in the output if there were spaces (or other ignorable tokens) in the normalized input 2017-03-15 03:35:03 -04:00
Al
d43989cf1c [fix] log_sum_exp in SSE mode shouldn't modify the original array 2017-03-15 00:22:17 -04:00
Al
c201939f3a [openaddresses] adding some of the new counties in GA. Adding the simple unit regex to DeKalb county's ignore list as there are a few in there 2017-03-13 16:08:37 -04:00
Al
e0a9171c09 [openaddresses] adding language-delineated files for South Tyrol 2017-03-13 01:27:23 -04:00
Al
6cf113b1df [fix] handle case of T = 0 in Viterbi decoding 2017-03-12 22:55:52 -04:00
Al
35ccb3ee62 [fix] move 2017-03-12 20:22:35 -04:00
Al
d40a355d8b [fix] heap issues when cleaning up CRF 2017-03-12 20:20:51 -04:00
Al
1277f82f52 [logging] some small logging changes to track vocab pre/post pruning 2017-03-12 00:24:52 -05:00
Al
7afba832e5 [test] adding the new tests to the Makefile 2017-03-11 14:34:27 -05:00
Al
7562cf866b [crf] in averaged perceptron training for the CRF, need to update transition features when either guess != truth or prev_guess != prev_truth 2017-03-11 05:58:23 -05:00
Al
a6eaf5ebc5 [fix] had taken out a previous optimization while debugging. Don't need to repeatedly update the backpointer array in viterbi to store an argmax when a stack variable will work. Because that's in the quadratic (only in L, the number o labels, which is small) section of the algorithm, even this small change can make a pretty sizeable difference. CRF training speed is now roughly on par with the greedy model 2017-03-11 02:31:52 -05:00
Al
647ddf171d [fix] formatting for print features in CRF model 2017-03-11 01:18:35 -05:00
Al
735fd7a6b7 [openaddresses] adding Douglas County, OR 2017-03-10 23:36:11 -05:00
Al
d876beb386 [fix] add CRF files to the main lib 2017-03-10 19:40:15 -05:00
Al
0ec590916b [build] adding necessary sources to address_parser client, address_parser_train and address_parser_test 2017-03-10 19:33:31 -05:00
Al
25649f2122 [utils] new_fixed and resize_fixed in vector.h 2017-03-10 19:31:34 -05:00
Al
4e02a54a79 [utils] adding file_exists to header 2017-03-10 19:30:46 -05:00
Al
5775e3d806 [parser/cli] removing geodb loading from parser client 2017-03-10 19:30:18 -05:00
Al
8deb1716cb [parser] adding polymorphic (as much as C does polymorphism) model type for the parser to allow it to handle either the greedy averaged perceptron or a CRF. During training, saving, and loading, we use a different filename for a parser trained with a CRF, which is still backward-compatible with models previously trained in parser-data. Making necessary modifications to address_parser.c, address_parser_train.c, and address_parser_test.c. Also adding an option in address_parser_test to print individual errors in addition to the confusion matrix. 2017-03-10 19:28:21 -05:00
Al
1bd4689c5f [openaddresses] add Gillespie County, TX 2017-03-10 15:53:49 -05:00
Al
171aa77ea3 [openaddresses] adding Fisher County, TX 2017-03-10 15:46:46 -05:00
Al
8e3bcbfc95 [openaddresses] adding Coffey County, KS 2017-03-10 15:44:59 -05:00
Al
b85ed70674 [utils] adding a function for checking if files exists (yay C), or at least the closest agreed-upon method for it (may return false if the user doesn't have permissions, but that's ok for our purposes here) 2017-03-10 13:39:52 -05:00
Al
3b33325c1a [cli] no longer need geodb setup in address parser client 2017-03-10 13:11:32 -05:00
Al
ef8768281b [parser/crf] adding runtime CRF tagger, which can be loaded/used once trained. Currently only does Viterbi inference, can add top-N and/or sequence probabilities later 2017-03-10 02:06:45 -05:00
Al
9afff5c9ed [parser/crf] adding an initial training algorithm for CRFs, the averaged
perceptron (FTW!)

Though it does not generate scores suitable for use as probabilties, and
might achieve slightly lower accuracy on some tasks than its
gradient-based counterparts like SGD (a possibility for libpostal)
or LBFGS (prohibitive on this much data), the averaged perceptron is
appealing for two reasons: speed and low memory usage i.e. we can still use
all the same tricks as in the greedy model like sparse construction of
the weight matrix. In this case we can go even sparser than in the
original because the state-transition features are separate from the
state features, and we need to be able to iterate over all of them
instead of simply creating new string keys in the feature space. The
solution to this is quite simple: we simply treat the weights for each
state-transition feature as if they have L * L output labels instead of
simply L. So instead of:

{
    "prev|road|word|DD": {1: 1.0, 2: -1.0}
    ...
}

We'd have:

{
    "word|DD": {(0, 1): 1.0, (0, 2): -1.0}
    ...
}

As usual we compress the features to a trie, and the weights to
compressed-sparse row (CSR) format sparse matrix after the weights have
been averaged. These representations are smaller, faster to load from
disk, and faster to use at runtime (contiguous arrays vs hashtables).

This also includes the min_updates variation from the greedy perceptron,
so features that participate in fewer than N updates are discarded at
the end (and also not used in scoring until they meet the threshold so
the model doesn't become dependent on features it doesn't really have).
This tends to discard irrelevant features, keeping the model small
without hurting accuracy much (within a tenth of a percent or so in my
tests on the greedy perceptron).
2017-03-10 01:28:31 -05:00
Al
5cac4a7585 [parser/crf] adding crf_trainer, which can be thought of as a "base class" as much as that's possible in C, for creating trainers for the CRF. It doesn't deal with the weights or their representation, just provides an interface for keeping track of string features and label names, and holds the crf_context 2017-03-10 01:25:20 -05:00
Al
dd0bead63a [test/utils] also a good thing to sanity check (in C especially): string handling code 2017-03-10 01:15:23 -05:00
Al
adab8ab51a [test/crf] test for crf_context, adapted from crf1dc_debug_context in CRFsuite. Always a good idea to sanity check numerical code 2017-03-10 01:13:40 -05:00
Al
f9a9dc2224 [parser/crf] adding the beginnings of a linear-chain Conditional Random Field
implementation for the address parser.

One of the main issues with the greedy averaged perceptron tagger used currently
in libpostal is that it predicts left-to-right and commits to its
answers i.e. doesn't revise its previous predictions. The model can use
its own previous predictions to classify the current word, but
effectively it makes the best local decision it can and never looks back
(the YOLO approach to parsing).

This can be problematic in a multilingual setting like libpostal,
since the order of address components is language/country dependent.
It would be preferable to have a model that scores whole
_sequences_ instead of individual tagging decisions.

That's exactly what a Conditional Random Field (CRF) does. Instead of modeling
P(y_i|x_i, y_i-1), we're modeling P(y|x) where y is the whole sequence of labels
and x is the whole sequence of features. They achieve state-of-the-art results
in many tasks (or are a component in the state-of-the-art model - LSTM-CRFs
have been an interesting direction along these lines).

The crf_context module is heavily borrowed from the version in CRFSuite
(https://github.com/chokkan/crfsuite) though using libpostal's data structures and
allowing for "state-transition features." CRFSuite has state features
like "word=the", and transition features i.e. "prev tag=house", but
no notion of a feature which incorporates both local and transition
information e.g. "word=the and prev tag=house". These types of features are useful
in our setting where there are many languages and it might not make as
much sense to simply have a weight for "house_number => road" because that
highly depends on the country. This implementation introduces a T x L^2 matrix for
those state-transition scores.

For linear-chain CRFs, the Viterbi algorithm is used for computing the
most probable sequence. There are versions of Viterbi for computing the
N most probable sequences as well, which may come in handy later. This
can also compute marginal probabilities of a sequence (though it would
need to wait until a gradient-based learning method that produces
well-calibrated probabilities is implemented).

The cool thing architecturally about crf_context as a separate module is that the
weights can be learned through any method we want. As long as the state
scores, state-transition scores, and transition scores are populated on
the context struct, we have everything we need to run Viterbi inference,
etc. without really caring about which training algorithm was used to optimize
the weights, what the features are, how they're stored, etc.

So far the results have been very encouraging. While it is slower to
train a linear-chain CRF, and it will likely add several days to the
training process, it's still reasonably fast at runtime and not all that
slow at training time. In unscientific tests on a busy MacBook Pro, so far
training has been chunking through ~3k addresses / sec, which is only
about half the speed of the greedy tagger (haven't benchmarked the runtime
difference but anecdotally it's hardly noticeable). Libpostal training
runs considerably faster on Linux with gcc, so 3k might be a little low.
I'd also guess that re-computing features every iteration means there's
a limit on the performance of the greedy tagger. The differences might
be more pronounced if features were pre-computed (a possible optimization).
2017-03-10 01:10:22 -05:00
Al
f9e60b13f5 [parser] size the postcode context set appropriately when reading the parser, makes loading a large model much faster 2017-03-09 14:31:12 -05:00
Al
2400122162 [fix] fixing up hash str to id template 2017-03-09 00:54:31 -05:00
Al
4c03e563e0 [parser] for the min updates method to work, the feature that have not yet reached the min_updates threshold also need to be ignored when scoring, that way the model has to perform without those features, and should make more updates if they're relevant 2017-03-08 15:40:12 -05:00
Al
a63c182e96 [parser] right context affixes need to use pre-normalized words as well 2017-03-08 13:51:36 -05:00
Al
ce9153d94d [parser] fixing some issues in address_parser_features. Prefix/suffix phrases use the word before token-level normalization (but after string-level normalization like lowercasing), needed to use the same string in the feature function as in address_parser_context_fill. Affects some German suffixes like "str." where the final "." would be deleted in token normalization, but the suffix length would include it. Also, three of the new arrays used in address_parser_context (suffix_phrases, prefix_phrases, and sub_tokens) weren't being cleared per call, which means computing the wrong features at best and a segfault at worst 2017-03-07 17:30:53 -05:00
Al
b6bf8da383 [utils] adding aligned malloc/free/realloc in vector.h and matrix.h, fixing bug in matrix_copy 2017-03-07 16:25:34 -05:00
Al
242b1364ae [parser] using new API in address_parser_test 2017-03-07 16:24:34 -05:00
Al
39f59e7ecf [openaddresses] adding Mayenne, FR 2017-03-07 15:41:33 -05:00
Al
c2b516c761 [openaddresses] adding Hernando County, FL 2017-03-07 15:11:53 -05:00
Al
749bb4907e [openaddresses] adding city of Carlsbad, NM 2017-03-07 10:55:09 -05:00
Al
154fd42299 [openaddresses] adding city of Amarillo, TX 2017-03-07 10:53:52 -05:00
Al
95015990ab [parser] learning a sparser averaged perceptron model for the parser using the following method:
- store a vector of update counts for each feature in the model
- when the model updates after making a mistake, increment the update
  counters for the observed features in that example
- after the model is finished training, keep only the features that
  participated in a minimum number of updates

This method is described in greater detail in this paper from Yoav
Goldberg: https://www.cs.bgu.ac.il/~yoavg/publications/acl2011sparse.pdf

The authors there report a 4x size reduction at only a trivial cost in
terms of accuracy. So far the trials on libpostal indicate roughly the
same, though at lower training set sizes the accuracy cost is greater.

This method is more effective than simple feature pruning as feature
pruning methods are usually based on the frequency of the feature
in the training set, and infrequent features can still be important.
However, the perceptron's early iterations make many updates on
irrelevant featuers simply because the weights for the more relevant
features aren't tuned yet. The number of updates a feature participates
in can be seen as a measure of its relevance to classifying examples.

This commit introduces --min-features option to address_parser_train
(default=5), so it can effectively be turned off by using
"--min-features 0" or "--min-features 1".
2017-03-06 22:28:33 -05:00
Al
5c1c1ae0f2 [parser] moving tagger function pointer definition to a separate header so it can be used for other models 2017-03-06 21:42:06 -05:00
Al
cc58ec9db2 [parser] fix another valgrind error in parser training (cstring_array memory can get moved around when using string pointers obtained before adding to it, which can potentially cause a realloc), no longer using the dummy START tags as the feature function can choose to add features for those cases 2017-03-06 21:39:14 -05:00
Al
754f22c79a [parser] moving feature printing to averaged perceptron tagger, taking advantage of trie prefix-sharing in feature incorporating previous tags 2017-03-06 20:32:50 -05:00
Al
839a13577d [parser] fixing affix-related valgrind errors in address parser features 2017-03-06 20:28:42 -05:00
Al
c3581557a1 [parser] counting classes instead of keeping a set 2017-03-06 20:05:01 -05:00
Al
a5283cb313 [fix] trie_new_from_hash 2017-03-06 15:57:42 -05:00
Al
23ed916f09 [openaddresses] adding Hattiesburg, MS 2017-03-06 15:45:23 -05:00