empress icon indicating copy to clipboard operation
empress copied to clipboard

sorted all menus

Open kwcantrell opened this issue 2 years ago • 10 comments

This sorts all drop down menus and the node select menu. This should hopefully make it much easier to find things :smile:

kwcantrell avatar Aug 12 '21 12:08 kwcantrell

The following artifacts were built for this PR: empire-biplot.qzv, empire.qzv, empress-tree.qzv, just-fm.qzv, plain.qzv

emperor-helper avatar Aug 12 '21 12:08 emperor-helper

The main motivation behind sorting everything came up while demonstrating EMPress to the health team during Tuesdays meeting. They have a lot of metadata columns and the current method made it fairly difficult to find things. I think for more general uses, this will be the case. It maybe nice to list confidence after taxonomy but I think that small convenience is outweighed by the possible frustration of having to search for every other category. I could be wrong though :smile:

kwcantrell avatar Aug 12 '21 21:08 kwcantrell

Fair enough! Since there's a compelling reason for it, I don't mind this change.

I think that small convenience is outweighed by the possible frustration of having to search for every other category.

Agreed.

We miiiight be able to get the best of both worlds by renaming Confidence to Taxonomy Confidence or something, which would automatically mean that the column would be listed after the taxonomy levels (the Level 1 names, etc. are already made up by Empress, so this has precedent). But that fix will cause its own problems and is a discussion for another day :)

fedarko avatar Aug 12 '21 22:08 fedarko

Looks like McHelper is failing again... :cry:

kwcantrell avatar Aug 26 '21 19:08 kwcantrell

@fedarko I believe I addressed all your comments.

kwcantrell avatar Aug 26 '21 19:08 kwcantrell

Did you try rerunning the jobs? I just restarted them. It’s likely an IP related problem(?). 🐧

Yoshiki Vázquez-Baeza

On Aug 26, 2021, at 12:51 PM, kwcantrell @.***> wrote:

 Looks like McHelper is failing again... 😢

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

ElDeveloper avatar Aug 26 '21 23:08 ElDeveloper

Yep, artifacts should be updated now.

ElDeveloper avatar Aug 27 '21 03:08 ElDeveloper

@fedarko any chance you can review and merge if things look good to you?

ElDeveloper avatar Oct 26 '21 15:10 ElDeveloper

@ElDeveloper yep, I will do my best to get to the remaining open pull requests ASAP.

fedarko avatar Oct 26 '21 19:10 fedarko

Thanks so much!

Yoshiki Vázquez-Baeza

On Oct 26, 2021, at 12:48 PM, Marcus Fedarko @.***> wrote:

 @ElDeveloper yep, I will do my best to get to the remaining open pull requests ASAP.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

ElDeveloper avatar Oct 27 '21 14:10 ElDeveloper