| Age | Commit message (Collapse) | Author |
|---|
|
|
|
It's a bit ugly. In that it uses the explicit string 'and' to avoid
creating those links. But that was the wording of the request.
In general we are missing an AST step. That would let us switch between
'#' and ' ' much more cleanly.
|
|
TODO: introduce a Node class split from the Graph.
TODO: use a GUID and not just an incrementing number (trivial change).
|
|
css:
use 0 instead of 0px (0 doesn't need a unit)
fallback rgb(r,g,b) for rgba
code:
missing semicolons
don't use initial caps for a function name
unused variables
|
|
|
|
|
|
|
|
to rewrite that and heavily test it)
|
|
chainlink temp node created from ''
|
|
|
|
|
|
|
|
|
|
|
|
In * case sentence - Description contains quote marks for multiple
worded sentence
|
|
|
|
broken by define addition (require.js)
|
|
Yes, I'm being lazy. The purpose of require.js is to allow writing
modules, i.e. individual smaller then the whole files with clear
dependencies. This is achieved by using a new api provided by
require.js:
define, require, requirejs
Read more at [requirejs] (http://requirejs.org)
Tests have been rewritten to work correctly with require.js; they still
use jsdom.
Test output is now stored in the repository, all tests are run via
run_tests.js; This is a hack - once a good harness (with assertions,
suites - unittest like) presents itself it will be used.
Hidden here is also using FileSaver for history saving, so history is
saved to history.json instead to a random name. Plus dropping some dead
code.
|
|
|
|
|
|
The whole lower case thing is a hack. We should separete id and name.
But right now it seems to be holding.
|
|
textanalysis, firefox sometimes lags
|
|
|
|
|
|
This reverts commit e58e0c0b520d4abb971d3917712340330e5d85ff.
It actually destroyed quite a lot of things, specifically even a simple
sentence ended up being a chainlink. I need to rethink the
'abnormalGraph' predicate.
|
|
Fixes #86
Well, almost - the center node contains the 'a and b and c' (with extra spaces too).
node test_analyzer.js '#a and #b and #c are cool'
digraph {
BUBBLE;
a;
b;
c;
"new node";
"a and b and c are cool";
a -> "a and b and c are cool";
b -> "a and b and c are cool";
c -> "a and b and c are cool";
}
|
|
|
|
This is more of #49 basically, just for a more complex sentence.
I've kept the flow that we have, but augmented the graph change check
(myGraph.compareSubset) and introduced some helper functions for sets on
the way.
|
|
|
|
|
|
|
|
Fixes #49
Note about solution: in general we need to find out whether the graph
from the new text (NewVertices, NewLinks) is homologous to the graph
from the old text (OldVertices, OldLinks). The implemented solution only
checks an easier case, where there is a single renamed vertex.
A better solution to do later:
- have textanalysis have state
- have textanalysis return the difference from the last state:
nodes added
nodes deleted
nodes changed
- plus for bonus refactor all the editing functions (edit* in myGraph)
|
|
|
|
|
|
|
|
|
|
|
|
TODO - code that doesn't make me choke up with tears
|
|
|
|
|
|
|
|
Previously:
prefix #a does #b => ((a, b), [(a, b, "does (prefix)")])
now:
=> ((a, b), [(a, b, "prefix #a does #b")])
|
|
|
|
|
|
|
|
|
|
1. move the sugg variable to textanalysis.js instead of the ui part
2. use a callback for the autocomplete widget (it supports three
options: array, string as url for a json returning service, or callback)
3. use a dictionary instead of an array to keep the list of suggestions.
We iterate on the dictionary instead of using a Trie or some other
datastructure in the callback. But the dictionary avoids iteration when
adding a word. Overall a loss in performance, but not noticeable.
Googling / memory suggests a Trie is what we want here, but I'm
postponing that for later. See:
http://ejohn.org/blog/javascript-trie-performance-analysis
|
|
|
|
|
|
|
