search

hypertrons / hypertrons-crx

A browser extension for insights into GitHub, Gitee projects and developers.
https://hypercrx.cn
Apache License 2.0
357 stars 102 forks source link

[Bug] Multiple tags can cause problems with tag arrangement #859

Closed wangyantong2000 closed 1 month ago

wangyantong2000 commented 3 months ago

Version

No response

Current Behavior

image

https://github.com/openai/whisper

image

https://github.com/opencv/opencv

Multiple tags can cause problems with tag arrangement, such as in the two repositories above.

Expected Behavior

No response

Environment

- OS:
- Browser:

Any additional comments?

No response

HalloMelon commented 3 months ago

I think when there are a lot of tags, some tags can be folded, as shown below:

image

wangyantong2000 commented 3 months ago

I think when there are a lot of tags, some tags can be folded, as shown below:

! [image](https://private-user-images.githubusercontent.com/79428895/357378619-b87e12f9-283c-4270-94a6-31d93d8dd503.png?jw t=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6I mtleTUiLCJleHAiOjE3MjM1NDYzNDYsIm5iZiI6MTcyMzU0NjA0NiwicGF0aCI6Ii83OTQyODg5NS8zNTczNzg2MTktYjg3ZTEyZjktMjgzYy00MjcwLTk0Y TYtMzFkOTNkOGRkNTAzLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJ TJGMjAyNDA4MTMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODEzVDEwNDcyNlomWC1BbXotRXhwaXJlcz0zMDAmW C1BbXotU2lnbmF0dXJlPWNmNWVlYTZjNmE4NzY1OTY2MTZlZGI4OWQ5Zjc2NDlmMDQ1ZDkyYjBkNzQ0MGMwMTgxM2ZmZWI5ZDNiODNlZDUmWC1BbXotU2lnb mVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.j6nOm1oXC90kAriBEl0HfE6jviZrKcwVwG_BFvMbH-g)

This idea may be feasible, you can give it a try.

frank-zsy commented 3 months ago

Is this the feature of v1.9.7 or later version? I am currently using 1.9.7 but can not see the labels.

wangyantong2000 commented 3 months ago

Is this the feature of v1.9.7 or later version? I am currently using 1.9.7 but can not see the labels.

This hasn't been released yet. Currently there is an issue that has been merged into the master branch of the repository.

wangyantong2000 commented 3 months ago

Because some repository label data is not only voluminous but also lengthy in text. One idea is to mimic the repository home page by adding a label for the repository underneath. Place red box 2 below red box 1. Might be worth considering.

未命名
frank-zsy commented 3 months ago

Because some repository label data is not only voluminous but also lengthy in text. One idea is to mimic the repository home page by adding a label for the repository underneath. Place red box 2 below red box 1. Might be worth considering.

Great idea, like it.

wxharry commented 3 months ago

Because some repository label data is not only voluminous but also lengthy in text. One idea is to mimic the repository home page by adding a label for the repository underneath. Place red box 2 below red box 1. Might be worth considering.

image

I am also thinking if it is possible to move the other labels to the end of the list, so users get to know what metrics they are reading.

frank-zsy commented 2 months ago

open_digger_labels

Here is a prototype of what mentioned in the comment. I think this looks really good and we can add hyperlink to corresponding leaderboard for these labels when we finished the labels leaderboards.

wangyantong2000 commented 2 months ago

Here is a prototype of what mentioned in the comment. I think this looks really good and we can add hyperlink to corresponding leaderboard for these labels when we finished the labels leaderboards.

This looks great.

TieWay59 commented 2 months ago

one more failing case: https://github.com/bytecodealliance/wasmtime

wangyantong2000 commented 2 months ago

Next, this task will be completed by @Xsy41 .

Xsy41 commented 2 months ago

Some labels are too long, the name is similar, how to deal with it? @frank-zsy failing case：https://github.com/openai/whisper

image
wangyantong2000 commented 2 months ago

The icon may be this one https://open-digger.cn/img/logo/logo-blue-round-corner.png ？

Xsy41 commented 2 months ago

This is the initial adjustment plan at present, let's see if there are any better suggestions.

image image
TieWay59 commented 2 months ago

Good job @Xsy41! LGTM. cc @tyn1998

Is it possible to use AI instead of the full spell? I guess we'll need a shortened set of tags in someplace. (and give the full form on some hover inform text.) cc @wangyantong2000

HalloMelon commented 2 months ago

Wonderful @Xsy41 , but I was wondering if it's possible to display a portion of the important tags and collapse the remaining tags, instead of collapsing all the tags directly. However, I'm not sure if there is a specific method to automatically select higher-priority tags or an algorithm to rank the tags by priority.

Xsy41 commented 2 months ago

Thanks to @HalloMelon and @TieWay59 . Currently we have two methods, the first method does not display the full name of the long label and displays it when the mouse is hovering. The second displays tags with higher priority, or some similar tags are not displayed repeatedly. As shown in the following figure, can the four labels in the middle that are not framed in red no longer be displayed?

image
wangyantong2000 commented 2 months ago

Thanks to @HalloMelon and @TieWay59 , at present we have two methods, the first method does not show the full name for long tags, and the mouse hover to show. The second shows tags with higher priority, or for some similar tags. As shown in the following figure, can you only show the following four labels framed in red?

image

At present, our label data is directly provided by OpenDigger. I think we can wait for @frank-zsy to reconfirm the label data and see if further processing is needed. See if it is changed by the data source itself or during development.

frank-zsy commented 2 months ago

The label structure in OpenDigger involves multiple layers, so one repo may has multiple labels in a technical area in different layers just like whisper. And In the label structure, the parent label name will be a prefix to its children labels.

I think only presenting the top-level label is neat and clean, but in the label data exported in OpenDigger, we do not provide if a label is a top-level label but a top-level label is prefix of its all children labels.

So there are 2 ways to fix this:

frank-zsy commented 2 months ago

Oh, actually I think the type of the label implies the level of the label. For Domain and Tech label, the type of the label will be Domain-n and Tech-n.

So just filter the labels with type Tech-n and n is not 0, and same to Domain-n, we can get rid of the children labels.

Xsy41 commented 2 months ago

Oh, actually I think the type of the label implies the level of the label. For Domain and Tech label, the type of the label will be Domain-n and Tech-n.

So just filter the labels with type Tech-n and n is not 0, and same to Domain-n, we can get rid of the children labels.

Thanks to @frank-zsy we filtered all xxx-n and n is not 0, and our final interface rendered the result.

image