Simplified the predicate candidate generation for Nom
Added dummy sentence creation for Nom during SemanticRoleLabeler init
Removed pre-processing during training (inside PreExtractor)
Changed system output to logger
Cleaned up pom.xml
Removed pom from .gitignore

Resolved a clash in lexicon naming by adding the parser view (also removed the "lexicon." prefix)

Replaced console output with logger calls
Updated compileModelsToJar to accommodate both Stanford and Charniak

Removed unnecessary dependencies
Switched to illinois-nlp-pipeline-0.1.2
Minor fixes

Fixed model deployment

fixed a bug that prevented the TextPreProcessor from being initialized outside SRL.

-Only works with the Lagrange inference solver for now

handle exception from getParsePhrase

Also supressed some compilation issues in curator-release.
Otherwise good to work. Gets F1 71.6

Migrated to Java 7
Removed JLISLearner (and related classes)

Using new AnnotatorService

Maven test pass

Shyam

Now uses illinoi-sl's cleaner interface and stanford parse instead of charniak. Moved to V5.1.4.

Also works without assertion.

See merge request !1

…anager

Conflicts:
	src/main/java/edu/illinois/cs/cogcomp/srl/SemanticRoleLabeler.java

numFeat threads as config param

Some issue with multithreaded feat ext. so this as a temp. soln.

See merge request !3

two config madness handled

Mvn test passes now. Test config modified.

See merge request !4

new models

all use 5.1.6!

See merge request !5

Solved the Configurator/ResourceManager bug when using the nlp-pipeline

Updated dependencies versions

Solved the Configurator/ResourceManager bug when using the nlp-pipeline

@mssammon **IMPORTANT**: this depends on both recent `illinois-ner` and `illinois-nlp-pipeline` updates I pushed; you will need to merge/deploy both, before merging this.

See merge request !6

Removed ISRLInference interface
Updated dependencies versions:
 - SemanticRoleModeler now extends the Annotator abstract class
 - Removed WordNet dep (using jwnl-prime)

Replaced with evaluator in core-utilities

…ring it more in line with other CCG NLP packages

…non-default parameters are loaded from that file; otherwise, all default parameters are used.

…Names.SRL_VERB etc.

Currently, this fails when I try to integrate SRL, with addView() throwing exception "View 'Verb' not found".

…to mssammon

Conflicts:
	pom.xml
	src/main/java/edu/illinois/cs/cogcomp/srl/SemanticRoleLabeler.java

updated to use new Annotator API, added Configurator classes

Updated to conform to updated Annotator API.  Added configurator classes to bring configuration in line with other CCG packages and make integration into pipeline more straightforward.
Note that I also updated to Gurobi 6.5.  This can be rolled back to 6.0 if preferred. 



See merge request !8

Removed wordnet from `.gitignore` since it's not getting created by `jwnl-prime`

Updated to latest dependencies

Removed wordnet from `.gitignore` since it's not getting created by `jwnl-prime`

See merge request !9

Removed the specification of the directory during `SRLManager` initialization
Removed WordNet-related initialization
Updated to latest dependencies

Now the models need to agree up to the minor version of the SRL releases (patch number shouldn't matter)
The models now have their own Maven `artifactId` to be easily deployable.

Updated to the latest dependencies
Using `DummyTextAnnotationGenerator` for the unit tests

The legal arguments/senses are contained in the resource files

Removed prop/nombank dependence during inference

The legal arguments/senses are contained in the resource files

See merge request !10

Now supporting `OJAlgo` ILP inference
Moved contents of `jlis` package to `learn`
Added pipeline properties

Added more pipeline properties
Squashed "weight vector loaded" message

Added OJAlgo as default solver and switched off the Curator

Windows support (+ minor refactoring)

Addresses multiple problems when running on Windows and w/o Gurobi
Fixes CogComp#8 and #2 

See merge request !11

Couldn't get it to work.

upgrade SL dependency



See merge request !12

Closes CogComp#9

Changed `CoNLLColumnFormatReader.LemmaIdentifier` to `PredicateArgumentView.LemmaIdentifier`