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

The item count in a group is not increased when dragging a entry to the group #4679

Closed
bernhard-kleine opened this issue Feb 20, 2019 · 7 comments

Comments

@bernhard-kleine
Copy link

bernhard-kleine commented Feb 20, 2019

JabRef 5.0-dev--snapshot--2019-02-19--master--1b6c7b5b9
Windows 7 6.1 amd64
Java 1.8.0_201

Steps to reproduce the behavior:

  1. When dragging a item into a group the item count does not increase. Only be opening the group one sees that the entry has been dragged at all, while the item count remains zero in the example
  2. ...
  3. ...
    grafik

Removing selected items from a group using right click on the group and use "remove selected entries from this group" is not working at all. I had to remove entries with right clicking the entry and using the "remove from group" dialog separately on each entry.

@AEgit
Copy link

AEgit commented Feb 21, 2019

This might be related to the following issues:
#3112
#2857

(see also: #1903)

@AEgit
Copy link

AEgit commented Feb 21, 2019

Can confirm first issue for:

JabRef 5.0-dev--snapshot--2019-02-20--master--3589dc3cb
Windows 10 10.0 amd64
Java 1.8.0_201

The second issue, however, is different for me:

Removing selected items from a group using right click on the group and use "remove selected entries from this group" is not working at all. I had to remove entries with right clicking the entry and using the "remove from group" dialog separately on each entry.

I am able to remove multiple entries from a group by selecting them in the main table, right-clicking on the respective group and using "Remove selected entries from this group". So, I cannot confirm the second issue.

@samiyac
Copy link
Contributor

samiyac commented Mar 21, 2019

Hi, I was able to reproduce this issue on my system and would like to work on resolving it.
However, I am only able to confirm first issue.

@CaptainDaVinci
Copy link
Contributor

CaptainDaVinci commented Mar 22, 2019

@samiyac the issue #4441 might be related as well (haven't tested it though). I've done some preliminary investigation on that issue, it might be useful.

@tobiasdiez
Copy link
Member

Thanks to @samiyac this should be fixed in the latest development version. Could you please check the build from http://builds.jabref.org/master/. Thanks! Please remember to make a backup of your library before trying-out this version.

@bernhard-kleine
Copy link
Author

Seems ok. Thanks a lot.

@AEgit
Copy link

AEgit commented Mar 29, 2019

I can confirm that his has been fixed in:

JabRef 5.0-dev--snapshot--2019-03-28--master--a73167933
Windows 10 10.0 amd64
Java 1.8.0_201

Note, however, that for large databases (in my case >16,000 entries) when performing such an action, i. e. either adding an item to a group or removing it, the CPU load shoots up to >80% on a Intel Core i7 Quadcore (incl. Hyperthreading). If you add several entries at once or consecutively to a group, it will take a couple of seconds until the item count is updated properly and the CPU load stays >80% for roughly 1 minute. So for large databases, the current implementation currently poses a performance problem.
I guess, the item count for all groups of the database is recalculated after each modification. Possible solutions to this:

  1. Recalculate only the item number of the modified groups, i. e. of those groups which gained one or multiple new items as member(s) or lost member(s).
  2. Allow hiding the item count for the groups panel and also do not calculated the item count in the background. This was possible in older versions - maybe it is also possible in the current development version, but I cannot find the appropriate setting. If this change is made, however, it should still be possible to see in the groups panel, whether an item belongs to a group or not. That means, that the group containing the item of interest should still be coloured/marked in the groups panel, but no item count should be display or calculated. So basically keep the rounded square and let it be coloured according to the groups item membership (as it is already) but do not display and calculated the item number.

I personally would already be happy with the second solution - which is probably easier to implement - since I do not really care about the number of items in a group, but I do care, whether the items that belong to a group are marked as such in the group panel.
If solution 2 is already present and I failed to find it, apologies for that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

5 participants