Update

Hi lauren, thanks for the updates!

Just a quick update so you know where I am, and some notes I have

-I changed the covariation color scheme to be initialized more similar
to the AnnotationGradientScheme and is now called "By RNA helices" in
the menu. I'm a bit worried that there are some bugs in there, but
I'm going to check for them after I finish writing the documentation.
I commented out the covariation scheme code in GAlignFrame,
AlignFrame, and PopupMenu. I kept it in since I'll probably add a
covariance color scheme similar to Rfam's in the future, but I'm
actually considering adding this as an option in a popup menu similiar
to the one that comes up for the "By Annotation" option.

We can talk about that. It might need a little storyboarding to work out what you need to have in the dialog box before you start coding it up.

-Changed the default coloring for the RNA helices color scheme to be
purine/pyrimidine so the sequence logo shows up

-jalview.utils.ColourUtil class has been checked into the repository

-Fixed the Group Colour menu bug - I forgot a "setText" for the
purine/pyrimidine scheme in PopupMenu.

:slight_smile: easily done! good work.

I cleaned up the code and wrote the help files. I've committed these
changes if you'd like to take a look. Instead of "RNA support" I put
"Nucleic Acid Support" although I haven't decided if this is better or
not.

I think 'nucleic acid support' is better - the documentation would get even more complex if we distinguish dna and rna too much !

One comment about the documentation - try to add in useful links to external resources where necessary - e.g. the WUSS notation, which will not be familiar to many Jalview users (the help viewer is a basic web browser, and most pages will display ok (if a little oddly) if they don't rely on Javascript). You could even embed a sample stockholm file to demonstrate the WUSS notation in action.

I'm going to write a summary of what I did and what is left
tomorrow and put it in the docs folder. Then I'll make a diff. It
looks like I won't be able to submit the diff until August 30, so we
don't have to stress about submitting this right now. I'll also be
finishing the nescent requirements tomorrow.

great!

I wanted to change the RNA helices colouring to be the same in the
annotation panel, but I couldn't figure out how to update the panel
after the "By RNA helices" coloring option was selected. I added a
"RNA helix" option to defining a type of secondary structure in the
annotation panel. I didn't get to spend much time fixing the
annotation diaplay for the WUSS notation -- I'd like to fix this
before you put my code into a new release, but unless you think this
is a quick fix, I won't try to fix it tomorrow.

We can talk briefly about this today if you want. I'm not sure if its a quick fix or not, so we can take a look at it together.

Do you think the "By RNA helices" coloring should only be available if
a Stockholm file was loaded?

I think it should only be available if there is rna secondary structure annotation on the alignment.. so there's an alignmentAnnotation object with a valid secondary structure pairs list that can be used to intialise it.

I'll try to be on at our usual meeting time tomorrow if you'd like to
discuss anything. Otherwise, do you want to meet Tuesday morning my
time?

Lets try and briefly meet Monday morning, just to check over the questions above. If we can keep it short, then I expect we'll probably need to meet again on Tuesday just to clear up any final issues you have before the pencils down date.

Jim.

···

On 15/08/2010 02:44, Lauren Lui wrote:
On 16/08/2010 09:11, Lauren Lui wrote:

--
-------------------------------------------------------------------
J. B. Procter (JALVIEW/ENFIN) Barton Bioinformatics Research Group
Phone/Fax:+44(0)1382 388734/345764 http://www.compbio.dundee.ac.uk
The University of Dundee is a Scottish Registered Charity, No. SC015096.

Oops. As some of you may have realised - this last posting was supposed to go to the development mailing list. Clearly I need more coffee :slight_smile:

Jim.

···

On 16/08/2010 12:19, Jim Procter wrote:

   Hi lauren, thanks for the updates!

--
-------------------------------------------------------------------
J. B. Procter (JALVIEW/ENFIN) Barton Bioinformatics Research Group
Phone/Fax:+44(0)1382 388734/345764 http://www.compbio.dundee.ac.uk
The University of Dundee is a Scottish Registered Charity, No. SC015096.

Hi Jim,

I was able to reverse the arrow for the closing part of the RNA
helices in the annotation panel. I added a picture to the help files
for RNA helices Coloring, but the quality of the picture is bad when I
open up help in Jalview. Should I make the size of the picture bigger
or increase resolution? It looks alright when I view it in eclipse. I
committed the changes to the help files and the picture, if you want
to take a look.

Thanks,

Lauren

···

On Mon, Aug 16, 2010 at 4:19 AM, Jim Procter <jprocter@compbio.dundee.ac.uk> wrote:

Hi lauren, thanks for the updates!

On 15/08/2010 02:44, Lauren Lui wrote:

Just a quick update so you know where I am, and some notes I have

-I changed the covariation color scheme to be initialized more similar
to the AnnotationGradientScheme and is now called "By RNA helices" in
the menu. I'm a bit worried that there are some bugs in there, but
I'm going to check for them after I finish writing the documentation.
I commented out the covariation scheme code in GAlignFrame,
AlignFrame, and PopupMenu. I kept it in since I'll probably add a
covariance color scheme similar to Rfam's in the future, but I'm
actually considering adding this as an option in a popup menu similiar
to the one that comes up for the "By Annotation" option.

We can talk about that. It might need a little storyboarding to work out
what you need to have in the dialog box before you start coding it up.

-Changed the default coloring for the RNA helices color scheme to be
purine/pyrimidine so the sequence logo shows up

-jalview.utils.ColourUtil class has been checked into the repository

-Fixed the Group Colour menu bug - I forgot a "setText" for the
purine/pyrimidine scheme in PopupMenu.

:slight_smile: easily done! good work.
On 16/08/2010 09:11, Lauren Lui wrote:

I cleaned up the code and wrote the help files. I've committed these
changes if you'd like to take a look. Instead of "RNA support" I put
"Nucleic Acid Support" although I haven't decided if this is better or
not.

I think 'nucleic acid support' is better - the documentation would get
even more complex if we distinguish dna and rna too much !

One comment about the documentation - try to add in useful links to
external resources where necessary - e.g. the WUSS notation, which will
not be familiar to many Jalview users (the help viewer is a basic web
browser, and most pages will display ok (if a little oddly) if they
don't rely on Javascript). You could even embed a sample stockholm file
to demonstrate the WUSS notation in action.

I'm going to write a summary of what I did and what is left
tomorrow and put it in the docs folder. Then I'll make a diff. It
looks like I won't be able to submit the diff until August 30, so we
don't have to stress about submitting this right now. I'll also be
finishing the nescent requirements tomorrow.

great!

I wanted to change the RNA helices colouring to be the same in the
annotation panel, but I couldn't figure out how to update the panel
after the "By RNA helices" coloring option was selected. I added a
"RNA helix" option to defining a type of secondary structure in the
annotation panel. I didn't get to spend much time fixing the
annotation diaplay for the WUSS notation -- I'd like to fix this
before you put my code into a new release, but unless you think this
is a quick fix, I won't try to fix it tomorrow.

We can talk briefly about this today if you want. I'm not sure if its a
quick fix or not, so we can take a look at it together.

Do you think the "By RNA helices" coloring should only be available if
a Stockholm file was loaded?

I think it should only be available if there is rna secondary structure
annotation on the alignment.. so there's an alignmentAnnotation object
with a valid secondary structure pairs list that can be used to
intialise it.

I'll try to be on at our usual meeting time tomorrow if you'd like to
discuss anything. Otherwise, do you want to meet Tuesday morning my
time?

Lets try and briefly meet Monday morning, just to check over the
questions above. If we can keep it short, then I expect we'll probably
need to meet again on Tuesday just to clear up any final issues you have
before the pencils down date.

Jim.

--
-------------------------------------------------------------------
J. B. Procter (JALVIEW/ENFIN) Barton Bioinformatics Research Group
Phone/Fax:+44(0)1382 388734/345764 http://www.compbio.dundee.ac.uk
The University of Dundee is a Scottish Registered Charity, No. SC015096.

_______________________________________________
Jalview-discuss mailing list
Jalview-discuss@jalview.org
http://www.compbio.dundee.ac.uk/mailman/listinfo/jalview-discuss

Hi lauren.

I'm not going to be able to meet today, so lets keep in contact by email. Sorry about that - I didn't check my diary properly when we were talking yesterday.

I was able to reverse the arrow for the closing part of the RNA
helices in the annotation panel.

That's great. I had a look at how you did this and thought of two things you might want to consider for the nescent demo version:
1. Using regexes is quite expensive - so it's better to use the standard string comparison functions when checking the content of the displayCharacter field of the alignment annotation Annotation object, since you don't really need the flexibility of the regex for this.
2. The redraw bug is due to you not adding in an 'S' clause for the switch statement further down the AnnotationPanel's rendering routine. Check the source carefully - you can pretty much copy and paste your rendering code into the second 'switch' statement, but there is an issue regarding getting arrow direction that you'll need to address to get it working right.

Thinking about this further, it might be better in the future to have a different symbol for the reverse strand - ie 'S' for 5->3 and 's' for 3->5 pairing. That would avoid messing around with the display character entirely. Ideally, it would be good to put in the raw WUSS notation instead - leaving the display character property free for adding labels (e.g. The kink-turn labelling which Kersten uses).

I added a picture to the help files
for RNA helices Coloring, but the quality of the picture is bad when I
open up help in Jalview. Should I make the size of the picture bigger
or increase resolution?

The problem is that the help viewer is a crummy browser - so it doesn't do a good job of rendering images at different scales. I would clip the image so it shows off the different helix colours and the stem annotation, without distorting the aspect ratio of the alignment (I think its a bit compressed currently - so you need to fiddle with the width and height attributed of the img tag to match the included image exactly).

Jim.

···

On 17/08/2010 00:32, Lauren Lui wrote:

Hi Jim,

I used regexps for the reverse arrow because I couldn't get the
comparison to work. I realize now that the '==' operator in Java
doesn't work how I thought it would yesterday, I should have been
using String.equals()! Anyway, I think using a different symbol for
the reverse strand is a great idea, so I'll change it. I'd still like
to be able to view the WUSS notation -- I haven't thought about how we
could still view this and leave the label's free for Kersten's
notation.

I'll fix the bugs and get the demo up on my blog.

Best,

Lauren

···

On Tue, Aug 17, 2010 at 7:51 AM, Jim Procter <jprocter@compbio.dundee.ac.uk> wrote:

Hi lauren.

I'm not going to be able to meet today, so lets keep in contact by email.
Sorry about that - I didn't check my diary properly when we were talking
yesterday.

On 17/08/2010 00:32, Lauren Lui wrote:

I was able to reverse the arrow for the closing part of the RNA
helices in the annotation panel.

That's great. I had a look at how you did this and thought of two things you
might want to consider for the nescent demo version:
1. Using regexes is quite expensive - so it's better to use the standard
string comparison functions when checking the content of the
displayCharacter field of the alignment annotation Annotation object, since
you don't really need the flexibility of the regex for this.
2. The redraw bug is due to you not adding in an 'S' clause for the switch
statement further down the AnnotationPanel's rendering routine. Check the
source carefully - you can pretty much copy and paste your rendering code
into the second 'switch' statement, but there is an issue regarding getting
arrow direction that you'll need to address to get it working right.

Thinking about this further, it might be better in the future to have a
different symbol for the reverse strand - ie 'S' for 5->3 and 's' for 3->5
pairing. That would avoid messing around with the display character
entirely. Ideally, it would be good to put in the raw WUSS notation instead
- leaving the display character property free for adding labels (e.g. The
kink-turn labelling which Kersten uses).

I added a picture to the help files
for RNA helices Coloring, but the quality of the picture is bad when I
open up help in Jalview. Should I make the size of the picture bigger
or increase resolution?

The problem is that the help viewer is a crummy browser - so it doesn't do a
good job of rendering images at different scales. I would clip the image so
it shows off the different helix colours and the stem annotation, without
distorting the aspect ratio of the alignment (I think its a bit compressed
currently - so you need to fiddle with the width and height attributed of
the img tag to match the included image exactly).

Jim.