attach annotations property to seq_span

make annotations sequenceable from python
e.g. l = [ x for x in mussa.annotations("value")
returns [('name', 'value')

basic wrapping of Annotations
this provides at least basic member access for my annotations class,
it unfortunately doesn't properly handle iteratation yet though.

more work on seq_span
One problem that showed up was that if a SeqSpan is created on the stack,
the parent shared pointer wont work, so any subseqs will crash.

To fix that at the python layer I had to add a factory function that
proprely dynamically created a SeqSpan.

I tried to get the parent call to work but that kept returning a new object
which makes the python is function unhappy.

translate KeyError too
unfortunately the template register_exception_handler had trouble with
function overloading of the translate functions, so I needed to name
each exception translator differently.

Start wrapping seq_span
there were a couple of challanges. First off the stl_container_adapter I
was using didn't handle returning a const_reference particularly well,
so I had to add a get_const function.

The other was more insideous, the example the wiki had showed just setting
the python exception, it didn't generate a C++ exception. So when
boost::python tried to copy an undefined reference when it fell of
the end of a container I got a bus error.

The solution was to add in a C++/Python exception translater so I threw
a C++ exception indicating an IndexError, but then the it was transformed
into the appropriate Python exception.

add prototype wrapping of the qt sequence browser
having two boost::python modules caused some python import problems
so i merged the gui code into the mussa python extension module.

Unfortunately it's a serious pain to create the right objects to be passed
to the browser for it to render anything. I need to switch the browser
to use the seqspans with drawables on them.

check for per python version libboost_python library
Ok, so this probably isn't useful for anyone but me yet, as I hacked
boost-cmake to add the python version number to libboost_python .

Though I will try to get boost.python to include it in the main branch.

Boost unit test 1.34.1 needs an extra define
The 1.33.1 version aparently tried to include main() in a shared
library, which is aparently doesn't work on every platform. So they
needed to add a macro which would expand in to a main() for
auto test cases.

Properites -> Properties spelling err fix.
 * Reported by Erich Schwarz.

HACKISH: Display chipseq peak window track
 * Displays chipseq peak windows when annotation is
   of type 'chipseq_peak_window'.
 * FIXME: A more proper implementation will be needed
   in the future.

remove excess cout call

make the mupa parser more robust
ticket:268
improved mupa parser

add threshold/window flag getter/setters

minor updates to mussa setup widget
I was trying to get mussa to build against qt 4.3.0 which and this component
had some display issues (which I wasn't able to resolve on OS X)
these updates remove the trailing blank space that was needed for qt 4.2.2
on windows (at least I think it was that version) and wrapped most
of the display strings in the tr() so we could localize mussa if we
wanted to.

update to 4.2.3

Use the right return type
Makes a type conversion warning go away.

test paircomp::NwayComparison
this does require paircomp revision 110 from
http://svn.idyll.org/svn/t/cartwheel/branches/diane/paircomp
if you're using paircomp.

Mussa will only try to build paircomp if there's a mussa/paircomp
directory.

Catch annotation sequences that don't end with newline
ticket:265
for some unknown reason if the last sequence in an annotation file
wasn't terminated by a new line, spirit ended up duplicating the
last character. Needless to say this meant that it wouldn't always
align properly.

I installed a hack into push_back_seq::operator() that checks for
this condition and removes the duplicated character if the bug was
triggered.

Needless to say if spirit (or my grammar) gets fixed and this bug goes
away the hack will need to be removed. I think the sequence equality
tessts in test_sequence::sequence_no_trailing_newline should trip if
the last character doesn't get duplicated.

Also I didn't test what happens if a location based sequence doesn't
end with a newline.

test annotating a sequence with fasta records in a stream
I split the load_annot function int one that opens the file and
one that reads the stream. (which is called by the file handling one)

This was so I could test some random sequence that Steven was having
trouble with. (which seems to be working correctly.)

FindBoost.cmake boost 1.34 Windows mingw patch

try to update for boost 1.34
I really need a better way of searching for all the different types of boost
names.

factor out viewport to display scaling functions

simplify zoom code
while describing my first stab at trying to simplify Brandons code to brandon
I mentioned the magic word "log" scaling. Which after some digging in
the depths of my memory produced a useful function for more simply scaling
the scroll wheel.

accept and ignore events
aparently when doing event processing in Qt you need to accept events
you handled.

PyQt compatible Python wrapping using SIP
  * First successful attempt at wrapping a mussagl Qt Widget in a way
that makes it compatible with PyQt.
  * Currently Wrapped:
    * MussaSetupDialog

Mouse scroll wheel zooming! =o)
 * ticket:252
 * MussaWindow & MussaAlignedWindows now zoom when using the mouse wheel.
 * SequenceBrowser emits mouseWheelZoom(double) when mouse whell use
   is detected.
 * SequenceBrowserWidget perculates the mouseWheelZoom(double) signal upward.
   * If you need to act on new mouse wheel zoom, you can connect to the
     signal at the SequenceBrowserWidget level.
 * Zoom automatically adjusts increment when switching form 1.0, 0.1, 0.01
   levels for smooth zooming.

dont draw sequence box when we can see sequence text
my new draw_track code didn't test to see if the sequence text was visible
thus hiding the text. This fixes that, though at the cost of adding
more logic in the process of rendering the display.

 * New line chars at EOF (missed one)
   * Magically make warnings disapear.
   * ticket:263.

New line chars at EOF.
 * Magically make warnings disapear.
 * ticket:263.

Load saved muway and set to muways soft threshold.

 * Fix for ticket:257
 * Updated nway code to save the soft threshold when saving.
 * Updated nway code to load the soft threshold when saving.
 * Updated MussaWindow.load code to use soft threshold loaded from muway file.
 * Upadted MussaWindow.load code to load properly when the window
   is empty to start with.
   * Added a reset() function to the ThresholdWidget to avoid emits
     * This actually didn't fix the problem (see ticket:262)
   * Broke the SIGNAL/SLOT loop causing multiple nways to be run, by
     breaking the connection from MussaWindow to ThresholdWidget while
     the ThresholdWidget.reset() function is called.

improve mupa comment character parsing
ticket:261
It also makes sure that load_mupa throws an exception when it loads
a file it doesn't understand.

I wonder if the GUI knows to catch this?

make mupa file loading eol-style insensitive
ticket:260
I refactored the load_mupa_file function into one that takes a file,
opens it, and then passes it to a different function that takes a stream.
AKA split a function in two, so I could more easily unit test the
mupa loading code.

Once it was unit testable, I moved multiplatform_getline out of sequence,
into its own new cpp file and changed the mupa loading code to use it instead.

Load N sequences in Mussa::load

Fix for ticket:240.

Mussa would not load the sequences because the .muway file would
declare that 0 sequences existed. Since the Mussa:load code needed to
know the number of sequences to load the sequences, it failed.

The new version of the load sequence code, reads the sequences in
until there are no longer any sequences to read. At that point, the
number of sequences is the_seqs.size() and therefore is no longer
dependant on the muway file.

escape clears a sequence browser selection
ticket:212

draw a grey background to a track

store default colors
to cut down on the problem of repeating myself, provide a place to store
a default gene and track color.

set annotation draw function by type
Additionally, I renamed a couple of functions to reflect that we're looking for valid sequence
instead of Ns, and my draw functions now use the color set in the drawable.
(Which also means that I have to set the color for the drawable class).

Mark real sequence instead of Ns
Since Mussa forces all annotations to be at least 1 pixel, marking where the real
sequence is located is visually clearer.

make the N whiteout a bit bigger

show where the Ns are
ticket:243

This also tests the new draw function pointer in the drawable class,
as the N marking code uses that functionality to highlight where
the undefined regions are.

There were some problems in making sure that a subseq had the
right parent as I needed the current parent for the proper
draw coordinates.

remove unncessary class qualifier
some versions of gcc dont like class qualifiers on function
declarations inside the class.

add function for marking 'N' regions
with the right draw function add_annotations_for_undefined_sequence
should be able to highlight the regions with Ns in them.

add draw function pointer to drawable

remove annot class in favor of SeqSpan
It's not quite as clean as I might like as my metadata Annotations class
is annoying to access in C++.

also to support our our "legacy" mussa annotation file format I added
a couple of annotation creating helper functions.

Since I didn't want to also change the motif class at the same time,
I just move the members of annot into motif. At some point I'll finish
getting ride of motif.

Add getdefault to Annotations
and dont use std::string& for parameters as a string reference
doesn't work with char * literals.

fix uninitialized pointer
ticket:256
I forgot to initialize the motif_list pointer in one of the Sequence
constructors. Unfortunately it was the one that was used by the mussa
load_sequence code, which hadn't been tested.

this patch adds a test, and fixes the reported problem.

fix problems with motif changes not showing up in sequencebrowser
This patch adds a couple of new unit test cases to make sure that
when the motif sequence list is changed it is actually reflected
in the sequence browser.

The problem was that when I originally coded it, a GlSequence held
a shared_ptr<Sequence> so even though SequenceBrowser had new copies
of GlSequence, the underlying Sequence was still shared.

The change to GlSequence being a subclass of Sequence with a copy
of the motif_list meant that changes to the motif_list in one
copy weren't reflected in the copy held by the SequenceBrowser.

I fixed it by changing sequence to hold a shared_ptr<list<motif> >.
the downside is that that the motif_list is now always shared
between copies of a Sequence, which is likely to cause problems
when opening a second mussa window.

However since I plan on tossing the current annotation and motif handling
code at some point in the near future, this patch should be "good
enough".

delete default layout in MotifEditor
Qt was generating warning messages about couldn't set layout
on the MotifEditor while one was already set. It turns out you
need to delete the default layout before calling setLayout().

test motif editor pointers
just make sure we have the right shared pointers in our editor

test motif editor
I was thinking this is where the source of my motifs disappearing
might have been, but aparently the motif editor does still work
correctly.

test MotifModel and MotifElement
still haven't found the bug where motifs aren't being rendered
on the GlSeqBrowser correctly.

seperate qt unittests into subdirectories
I wanted to move the various qt unit tests into subdirectories to
make it a bit cleaner to add more tests.

incorporate drawable and annotations
this removes most of the contents of GlSequence in favor of drawable,
and adds shared_ptrs to both drawable and annotations into SeqSpan.

One problem is getting the code to do coping instead of sharing a
reference at the right time.

What I've got right now is I added some constructors that take ThingRefs
(aka boost::shared_ptr<thing>) and those will create copies, while
the more standard reference or pointer initializers just copy the
shared_ptr.

This was a bit problematic in the case of GlSeqBrowser, which really
needs to make sure that its glsequences have their own drawable component.

create class to hold drawing information
this will be added to SeqSpan to add information needed to draw
stuff on the screen--only if a particular SeqSpan is being rendered.

create a metadata holding class

Handle subseq when the parent is on the minus strand

move strand into seqspan
this patch moves strand storage from Sequence into SeqSpan,
also when accessing sequence data from a SeqSpan that is currently
on the minus side it correctly returns reverse complemented data.

Unfortunately getting iterators that point to data thats a
transformation of currently existing data is a giant pain and
was making SeqSpan way to complicated.

So I took the easy way out. SeqSpan will now cache a reverse complement
of the region its pointing to and return iterators based on that.

Also Alphabet can reverse complement a string,

Drat, just realized that if one changes the region of SeqSpan,
the rc_seq will need to be invalidated.

Missed part of "Store Sequence sequence location in a shared_ptr class"

Bad things happen if you let eclipse refactor your _darcs pristine
repository. Modifications that you made don't get recorded.

This patch should make buildbot happy.

Move alphabet type into SeqString
This is another large patch.
This moves alphabet_ref from Sequence to SeqString.

I still need to deal with a better way of handling reverse complementing
especially given my attempts to keep everything in reference counted
structures.

Also this patch and the previous:
Store Sequence sequence location in a shared_ptr class
have broken the display of motifs in mussa.

(I used to be using shared_ptr<Sequences> for everything,
but now since there's a couple of seperate objects updating
the motifs in one doesn't really help so much for the other.

so warning, these two patches and the next few that I make
are going to be unstable.
**END OF DESCRIPTION***

Place the long patch description above the ***END OF DESCRIPTION*** marker.
The first line of this file will be the patch name.

This patch contains the following changes:

M ./alg/CMakeLists.txt +1
M ./alg/alphabet.cpp +35
M ./alg/alphabet.hpp -1 +14
M ./alg/motif_parser.cpp -3 +4
A ./alg/seq.cpp
M ./alg/seq.hpp -3 +17
M ./alg/seq_span.cpp -6 +3
M ./alg/seq_span.hpp -1 +4
M ./alg/sequence.cpp -82 +26
M ./alg/sequence.hpp -15 +10
M ./alg/test/CMakeLists.txt +1
M ./alg/test/test_alphabet.cpp +5
A ./alg/test/test_seq.cpp
M ./alg/test/test_seq_span.cpp +9
M ./alg/test/test_sequence.cpp -43 +43
M ./qui/motif_editor/MotifElement.cpp -2 +2

Store Sequence sequence location in a shared_ptr class
This is a large (176 changes) patch, that moved most of the
information needed to store a sequence location into a class called
SeqSpan, which is designed to be solely managed as a shared_ptr.

This is one step in converting sequence annotations into being tree
of subsequences.

The hope is that the Sequence class doesn't store very much and
is mostly responsible for managing various collections of shared_ptrs
(which are a bit annoying to work with as local variables).

This also updated GlSequence to be a subclass of Sequence
(instead of holding a reference a shared_ptr<Sequence>)

Those changes required updating many random little function
calls.

make sure paths continue to start with a /
for some reason os.path.split was loosing the leading / for some
of my paths, this tests to see if it was there originally and
if it was, makes sure that its still present.

add a useful comment

Sub-analysis failure to draw on init fix (ticket:254)
 * This patch triggers a redraw after the window has been created by setting the dynamic threshold to the minimum.

Save to image crash fix (ticket:251)

mupa for test case
this is used to verify that the mussa tosses a useful exception
when someone sets a bad filename for an annotation.

cmake was trying to link -loptimized
ticket:250
aparetnly QT_QTCORE_LIBRARY wasn't defined, so cmake was trying to link
-loptimized. However QT_QTCORE_LIBRARY_RELASE was defined, which
makes CMake happier

Mussagl version increased to 1.1.0

WhatsThis update 2
 * Removing '*' since the bug seems to be Linux specific
 * Just having a space at the end fixes the linux whatsthis.

WhatsThis update

Added WhatsThis comments for:
  * Sequence info bar
  * New analysis dialog box

Added Qt version to Mussagl about box.

keep motifs after closing window
ticket:195
My previous implementation kept destroying and recreating the motif
editing dialog box. This version only deletes the motif editor when
MussaWindow.clear() is called.

more error messages for loading an annotation.
ticket:239
Return specific error messages if a path doesn't exist or is a directory
instead of a more generic there was a problem loading.

ignore paircomp
currently don't pay attention to the subversion checkout of paircomp
I have

Added newline char at end of file to remove gcc warning

Added newline char at end of file to remove gcc warning

Added new line at end of file to remove gcc warning

Added newline char to end of line (remove compile warning)

IsFileValidator doesn't use signal/slots

detect 64bit processors and add -fPIC
this is an intermediate solution as I probably can use ADD_DEFINES()
to add the command line options for the compiler instead of fiddingly
with CMAKE variables

Props to Brandon for finding the solution in an openwengo patch.
http://dev.openwengo.org/trac/openwengo/trac.cgi/changeset/9139
Long Live CMake!

add some newlines at the end of some of our files
gcc 4.2 complains about this.

conditionally build the paircomp tests

paircomp used the string find_first_not_of
so i needed to add it to my sequence class. (I'm trying to keep
my sequence class to look like an immutable string).

split unittests into seperate executables
ticket:208
I decided it was easer to debug my unittests when they were
in smaller pieces.

build paircomp if available

len(mussa.paths()) should equal the number of found paths
ticket:244
for some reason I was returning the number of sequences instead,
that is available as sequence_count.

Renamed size_t in NwayPaths to NwayPaths::size_type

Also I exported the various types of path size counts to the python layer

be resistent to not creating the browser
just on the off chance that MussaWindow.browser isn't created
test the pointer before trying to call it.
(AKA some left over debugging code)

make mussawindow initialize icons
this is also part of the useless try to stop qobject::moveToThread
warnings, but I decided it was a good idea for when qui/mussagl.cpp:main
isn't called because we were launched from a python extension.

try to reduce number of moveToThread warnings
I went off on a fools errand of trying to get rid of the QObject::moveToThread
warnings that Qt was spitting out when trying to prepare an icon.

Qt does prefer widgets to be dynamically constructed and to have their
parent widget set, which is what I tried to do to cut down on the
useless chatter.

(Note the chatter only happens on OS X when the frameworks have been installed
into the application bundle).

fix infinite loop for absolute pathnames
and add a --prelink-only update
and update to qt 4.2.2

provide get_py and get_py_ptr
I didn't like (*get_py)['name' as a way of getting a python namespace
object. So I changed get_py back to returning a reference by introducing
a new function get_py_ptr which creates the python singleton and returns
a pointer to it.

cut down on unnecessary linking to the python library

Allocating objects in the constructor causes problems
What do you know, being lazy and initializing Qt GUI objects in the
constructor cause problems. (AKA a segfault when running as a test case).

This patch allocates the various GUI objects in an actual function
(which is so the different contstructors can use the same initialization
code.

It looks like another coding standard to remember.

make compiled in python extension initialization more flexable
Because of weird library ordering issues, my test case that tested
running python code was looking for compiled in python modules that
weren't available yet it wouldn't link properly.

I solved this by making it possible to provide a list of what
compiled-in python extensions should be initialized when you launch
the interpreter, so the mussagl program, which does have all the right
components linked in is what is specifying what modules should be
provided to the python interpreter.

Now I just need to fix the build system so you can compile mussa without
all this python crud if needed.

And maybe decide how to launch the gui from a stanard python interpreter.

directly build the python extension modules
I learned about SET_TARGET_PROPERTIES( ... PREFIX )
to change the starting prefix for a library, which means
that I don't need to copy the libX.so to X.so to make something
a python importable module.

provide python interpreter for mussa qui via a seperate thread
helps ticket:201
Since both the python read-eval-print loop and Qt have their own
event loops one can't easily have a single threaded application
provide GUI code while still having an interpreter sitting around.
Additionally due to the limitations of OS X and Windows, the main
thread needs to be the GUI thread, so I needed to start the python
interpreter in a seperate thread.

I implemented several helper classes in qui/threading to help manage
creating threads. Currently it just provides a singleton interpreter
thread. The interpreter when started in its subthread will try to
initialize ipython, and if that fails will try InteractiveConsole
(I suppose I should've caught that failing, and then finally fell back
to the fgets read-eval-looop).

The way access to the GUI layer is handled is via a signal/slot thread
specific proxy class which holds a mussa analysis object and notifies
the copy in the main thread via a signal to create a new window using
the provided mussa object.

Access to some parts of the mussa object are thread safe, because I was
so enthusiastic about marking functions as const. All the other parts
are, shall we say, "up for grabs".

The other issue is I really need to improve the CMake scripts to be
a bit more robust about deteriming if python should be linked in.
(And one of my tests breaks because of this patch).

But since it works on OS X I want to know what it'll do on linux.
I'm pretty sure since a windows GUI app doesn't have a console that
it won't work on windows. (To say nothing of the command line
argument parser crashing under windows).

allow adding and viewing sequences to an analysis from python
this required a way to deal with the collection of sequences
returned by Mussa::sequences()

the python wiki had some suggestions (though their code didn't compile)
http://wiki.python.org/moin/boost.python/StlContainers

add python interface to seqcomp
which is actually provided by the class FLP

Finish off setting sequence offset user interface
ticket:111
I thought it would be better for the edit sequence properties action to live
under the main edit menu, instead of the popup menu.

Also I decided to use apply/close instead of ok/cancel as it seemed better
to make fiddling with the offset interactive.

And yes I probably should implement some graphical way of aligning the
sequences instead of just typing in offsets.

quick option to show sequence properties window