search

geneontology / amigo

AmiGO is the public interface for the Gene Ontology.
http://amigo.geneontology.org
BSD 3-Clause "New" or "Revised" License
29 stars 17 forks source link

Icons need to be deterministically generated #344

Open kltm opened 8 years ago

kltm commented 8 years ago

Take a look at [Neighborhood] here: http://tomodachi.berkeleybop.org/amigo/term/CL:0000540

There are a large number of relations with no icons, looking a little icky. Rather than depend on the icons being where they "should" be, they should be generated on the fly from simple parts. Colors would come from the context, the letter just the initial (or some such).

kltm commented 8 years ago

No need for these to be images in the future, they could just be text blocks and colors.

cmungall commented 8 years ago

or for now, default to a single label if no icon available

On 11 Apr 2016, at 9:37, kltm wrote:

No need for these to be images in the future, they could just be text blocks and colors.

You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geneontology/amigo/issues/344#issuecomment-208438834

kltm commented 8 years ago

Can't: it's not stored in any metadata. Rather than start storing that, I'd just go ahead and create on the fly. Weird thing to store.

cmungall commented 8 years ago

I suggest reframing this ticket.

I think the action here is to change neighborhood view to simply show the relation label. OLS beta gets this view right (but note that they don't load go-plus, so not all relations are visible. The point is that if OLS were to load go-plus it would show things in a sensible way). I'm of course talking about the Term Relations tab, not the wiggly octopus.

Icons are only a good idea for a smaller set of relations, such as those it makes sense to have in a graphical or drill-down hierarchical display. We should have manually crafted nice looking icons for these. For power-user displays that show unrestricted axioms in a tabular to textual form, text is fine.

kltm commented 8 years ago

Some offline talk with @cmungall . There is also the use case here with drill-down and the "looking up" browse views (as well as others) that have the irreducible chains that would also benefit from some autogeneration.