nfr-autochain-performance - Sophie autochaining is very slow when working with large texts.

[deyan]

it's taking me at least 5 seconds to turn pages in a small book (500x300) with auto-chained text on the pages.

• Sophie autochain is too slow for big text, improve the performance of the autochaining.

[deyan]

Batch update from file report_1.csv

[deyan]

Batch update from file 0911261.csv

[mira]

There are two basic problems:

1. There is a cyclic dependancy between the view and the model.
2. While mass changes like creating many pages, frames and so on all properties of those views are updated.

The first problem could be solved if the TextFrameH does not take a BookView as argument and does not need for views to be constructed to get all frames in a chain.

The second is a bit more complicated - we would need a single property that keeps all
chains in a book and does not update while mass updating - just returns its old value until chaining stops.

• Make an immutable class ChainInfo that simply wraps the ImmMap<ResourceRef, ImmList<TailTextH>>. Its should proivde a method for getting a chain based on the ResourceRef of its head-frame. It should also provide hashCode and equals methods so that when there are no changes in the chains, the view properties do not recompute.
• Make a class called ChainMaster. It would keep the chains for a single book. It would have final field keeping BookH for this book. It should also have an AutoProperty chainInfo() that maintains an acurate list of all chains in this book.
• the most significant delay in chaining methods is the updating of the view properties when making series of changes over the model. So there should be a property that keeps a boolean value indicating that those props should not be recomputed. While it is true the chainInfo() property will just return its last value.

Set this property to true always when there are a lot of changes expected and be shure to set it back to false after.

[mira]

The implementation is made according to the design.
There was added one more thing for general perfomance - the equals and hashCode methods are made to work with parts instead of ids. In order to do this I made sure that there were no "" elements in the parts list.

There is a changeset attached to this ticket.

[mira]

change-set

[meddle]

So, you are very absent minded and leave so things like:

• Bad spacing
• System.outs
• Comments that have no meaning in the trunk...

Be more careful in the future:

• Changed the ChainMaster to ChainManager, you know why :)
• I strongly recommend you investigate the two or more chaining bad performance... Speak with deyan or todor to write you a ticket, if they do not want kick them in the teeth!!! :)

You didn't score the analysis... bad bad bad...
Fill your fields!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

3.5p

[meddle]

Closing all the tickets before M Y1