Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sort labels in alphabet order #34

Closed
newpavlov opened this issue Aug 17, 2018 · 11 comments
Closed

Sort labels in alphabet order #34

newpavlov opened this issue Aug 17, 2018 · 11 comments
Assignees
Labels
Easy to fix The issue is easy to fix and probably it will be release in a next minor release enhancement New feature or request
Milestone

Comments

@newpavlov
Copy link

Currently they are sorted by primary key. If all labels were provided at task creation it results in a semi-random order of labels. It makes significantly harder to find required label when working with a big number of them. One work-around is to add labels one-by-one, but it's not a pleasant process...

@nmanovic nmanovic self-assigned this Aug 17, 2018
@nmanovic nmanovic added enhancement New feature or request help wanted Easy to fix The issue is easy to fix and probably it will be release in a next minor release labels Aug 17, 2018
@nmanovic nmanovic added this to the Release 0.2.0 milestone Aug 17, 2018
@nmanovic
Copy link
Contributor

Hi @newpavlov ,

Thank you for your feedback. I agree that it isn't convenient to search labels if they are not ordered. I believe it is possible not only to have them in alphabet order but also type on keyboard label's name if the input element in the focus.

Please let me know if you can send us a PR with the feature. We will really appreciate that.

@nmanovic nmanovic modified the milestones: Release 0.2.0, Alpha 0.3.0 Sep 25, 2018
@newpavlov
Copy link
Author

Unfortunately it can't be done on the back-end and should be done on front-end side. (server returns unordered dictionary which is processed by JS) At first glance I was unable to find the relevant JS code.

@ppavlovdev
Copy link

ppavlovdev commented Oct 19, 2018

Hi @newpavlov @nmanovic ,

I would suggest to rewrite list responses logic django-filters.

Here is the great filtering/searching functionality that we can combine with AJAX for real-time interaction.

Example request:
GET /labels?order_by=label_name

UPD:
In fact, we can rewrite not only this part of app. I think it's better to make all lists such flexible.

@nmanovic
Copy link
Contributor

Hi @cptmurphy ,

Thanks for your advice. We will look at the functionality.

@ppavlovdev
Copy link

@nmanovic,

Please, let me know when you make a decision. I would like to participate in your project.

@nmanovic
Copy link
Contributor

@cptmurphy ,

If you would like to prepare the PR we will be glad to access it. Just assign the issue on yourself. How much time do you think the issue can take?

@ppavlovdev
Copy link

@nmanovic ,

I need about a week to get acquainted with your codebase, then I can tell how much time it'll take.

@nmanovic
Copy link
Contributor

@cptmurphy ,

It will work for us. Let me reschedule the feature on beta. Thanks for your help and support.

@nmanovic nmanovic modified the milestones: Alpha 0.3.0, Beta 0.3.0 Oct 23, 2018
@ppavlovdev
Copy link

Hi @nmanovic,

Sorry for delaying, I was busy for a while.

What about update: It doesn't seem that I could integrate django-filters in current app's architecture. Let me explain:

@snosov1
Copy link

snosov1 commented Jan 10, 2019

Should this be closed now due to #242 ?

@nmanovic
Copy link
Contributor

@snosov1 ,

I think yes. Even it is possible to sort labels on client side as well (in case the task was created with unordered list of labels) I don't think that it makes sense for now.

TOsmanov pushed a commit to TOsmanov/cvat that referenced this issue Aug 23, 2021
* Fix default arg in voc converter

* Fix empty dataset export in mots
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Easy to fix The issue is easy to fix and probably it will be release in a next minor release enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants