Single Precision DVector #583
Conversation
floats internally. It exposes and accepts all numbers as doubles.
…dently of each other.
…eepWSinglePrecision
…gative number, so we can distinguish single precision data from the older double precision data.
and conver to a csv.
…int, feature pruning, and improved learning rates and thicknesses as the result of a parameter sweep.
| @@ -62,7 +62,17 @@ public SpanLabelView(String viewName, String viewGenerator, TextAnnotation text, | |||
| public void addConstituent(Constituent constituent) { | |||
| super.addConstituent(constituent); | |||
|
|
|||
| Collections.sort(this.constituents, TextAnnotationUtilities.constituentStartComparator); | |||
There was a problem hiding this comment.
is it worth simply replacing the map with a TreeMap, as you suggest? if not now, please open an issue.
There was a problem hiding this comment.
This is an inherited data structure. There is already an issue open for this. I will revise the existing issue.
| # | ||
| # first build cogcomp-nlp. Oddly enought, take much long to build than to |
There was a problem hiding this comment.
just to show I'm paying attention :-)
| # | ||
| echo | ||
| echo -------- cloning code into new directories -------------- |
There was a problem hiding this comment.
seems like this will take up a lot of space... we may want to think about how to refine it. Is it plausible to simply change output paths, or at least just create symlinks -- I'm assuming the run script launches separate processes for each parameter set; even in parallel, would this create any problems?
There was a problem hiding this comment.
I already know how to refine this, but way out of scope for this release. This is already much smaller that it was in the previous iteration. The right way to do this is to run them all in one VM. Makes the hairs on the back of my neck rise. Much smaller memory footprint, much faster, lots less disk.
| */ | ||
| public static void main1(String[] args) throws IOException { | ||
| /** get the F1 score for token only. */ | ||
| String fuck = "Token-level Acc Level1:\n" + |
There was a problem hiding this comment.
While guilty of the odd obscenity myself in comments, maybe rename this variable?
There was a problem hiding this comment.
hehehe. Whoops. that was a test method and should have been removed.
|
Looks good to me. |
|
Good to merge and deploy. |
The DVector now stores floats, can read floats or doubles, but only writes floats.