From TheITGuy...@gmail.com on February 06, 2013 06:17:34

Of note, cacheEdges() does not trigger an update of the cached edges.

From HDie...@lbl.gov on February 07, 2013 09:09:19

There is now a new method clearCachedEdges() in OWLGraphWrapper, see r1122 .

Please, can you verify that this is sufficient for your tasks.
Are there any other caches you want to be able to clear?

Status: Started

From TheITGuy...@gmail.com on February 07, 2013 15:16:45

It does not seem to work :/

Of note, my previous sample code was incorrect, to reproduce the problem, getEdgesBetween must have been called before the creation of a new relation.

So, by introducing the new clearCachedEdges() method, the sample code becomes:

myWrapper.getEdgesBetween(myClass, myClass2);//just to trigger the cache

OWLSubClassOfAxiom ax = factory.getOWLSubClassOfAxiom(myClass, myClass2);
AddAxiom addAx = new AddAxiom(myWrapper.getSourceOntology(), ax);
myWrapper.getManager().applyChange(addAx);

for (OWLGraphEdge edge: myWrapper.getEdgesBetween(myClass, myClass2)) {
System.out.println(edge);//no edges found here
}
myWrapper.clearCachedEdges();//test the new methd
for (OWLGraphEdge edge: myWrapper.getEdgesBetween(myClass, myClass2)) {
System.out.println(edge);//still nothing :(
}
//force an update, but throw checked Exceptions, and looks like a hack.
myWrapper = new OWLGraphWrapper(myWrapper.getSourceOntology());
for (OWLGraphEdge edge: myWrapper.getEdgesBetween(myClass, myClass2)) {
System.out.println(edge);//the edge is correctly displayed
}

From HDie...@lbl.gov on February 07, 2013 16:11:11

Thank you for testing.
I have updated the method to clear all caches and added a test case.
Please, see revision r1131 .

From TheITGuy...@gmail.com on February 07, 2013 16:21:13

Now it works, thank you.
(not a priority, but an option to disable the caching would be handy, when you build/modify an ontology)

From HDie...@lbl.gov on February 07, 2013 16:27:43

Hello,

there is the Config object for OWLGraphWrapper#getConfig()
It has the boolean field: isCacheClosure You could set that to false.
BUT at the moment this is not used in all of the caching methods. This needs to be changed soon.

From TheITGuy...@gmail.com on February 07, 2013 16:30:02

Oh OK, thank you for the info. I guess I will use clearCachedEdges() until then.