search

MGX-metagenomics / MGX-client

GNU Affero General Public License v3.0
0 stars 0 forks source link

Group name handling in clustering module #46

Closed sjaenick closed 7 years ago

sjaenick commented 7 years ago

In gitlab by @sjaenick on Sep 24, 2015, 18:16

Assigning a group name of ")))" triggers an internal error, most likely due to newick parsing.

TODO: add regression test + fix.

sjaenick commented 7 years ago

In gitlab by @sjaenick on Nov 10, 2015, 20:43

remove group names, use uuid instead and replace by original name later on

sjaenick commented 7 years ago

In gitlab by @sjaenick on Nov 23, 2015, 14:34

2 levels of obfuscation here: groups mapped to their uuid, and uuids are mapped to an internally generated name (this is what is sent to the server); current implementation reverses uuid->generatedName, but uuids remain in the returned newick string.

ToDo: Change API to return TreeI.

sjaenick commented 7 years ago

In gitlab by @sjaenick on Mar 24, 2016, 15:56

mentioned in commit cb352d1679e9f0ee06f854d7afd8eafa5dd2bc12

sjaenick commented 7 years ago

issue refers to different component