thechiselgroup / biomixer

BioMixer
http://bio-mixer.appspot.com/
16 stars 13 forks source link

Cycle in Tree #511

Open everbeek opened 9 years ago

everbeek commented 9 years ago

I refocused onto "anatomical structure", then expanded concepts for that node, allowing all of the nodes to be loaded. Then I clicked undo, then redo. The nodes go to the top right corner, and errors in the console indicate a recursion problem.

http://127.0.0.1:8888/conceptPathToRoot.html?initial_vis=paths_to_root&ontology_acronym=UBERON&full_concept_id=http://purl.obolibrary.org/obo/UBERON_0006590

everbeek commented 9 years ago

That commit above is for #510, not this issue.

everbeek commented 9 years ago

Oh, right, I see that I hadn't merged Elena's work yet. Doing so, and it will probably fix this problem. Therefore, likely dupe of #489.

elenavoy commented 9 years ago

I haven't finished dealing with cycles yet because I got caught up with my thesis. I think I'll be working on Biomixer in April, so I'll be able to fix that.

everbeek commented 9 years ago

Ok, I will not merge yet.