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.

Sign up for GitHub

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

Jump to bottom

Performance issues in development version 5.0 #4430

Closed
AEgit opened this issue Oct 29, 2018 · 4 comments
Closed

Performance issues in development version 5.0 #4430

AEgit opened this issue Oct 29, 2018 · 4 comments
Labels
bug Confirmed bugs or reports that are very likely to be bugs groups maintable type: performance
Milestone
v5.0

Comments

@AEgit
Copy link

AEgit commented Oct 29, 2018 •
edited
Loading

JabRef 5.0-dev--snapshot--2018-10-28--master--05047f32a
Windows 10 10.0 amd64
Java 1.8.0_191

Steps to reproduce the behaviour:

  1. Open a large database (in my case > 16,000 entries)
  2. Select a group in the groups panel
  3. Entries belonging to the group are displayed in the main panel
  4. Select another group in the groups panel
  5. Entries belonging to this group are displayed again in the main panel. Note, however, that there is a noticeable delay until the entries are displayed (about 1 to 2 seconds).

If you repeat steps 4 and 5 you will always notice this delay of one or two seconds. This can get quite annoying if you quickly have to change groups. Note, that in JabRef version 3.8.2 (which I am still using as my main version) there is no such delay. Is it possible to get a performance in handling large databases in version 5.0 that is comparable to that of 3.8.2?
@tobiasdiez tobiasdiez added bug Confirmed bugs or reports that are very likely to be bugs groups maintable labels Oct 29, 2018
@Siedlerchr
Copy link
Member

The changing of group is still slow and causes a spike in CPU load. Tested with a large database o 6,400 entries and many groups.
Memory is not an issue.
grafik

@AEgit
Copy link
Author

AEgit commented Jun 20, 2019

Can confirm - I might have time during this weekend to run some additional performance test. Unfortunately my YourKit Java Profiler test license ran out, so I will probably have to create a new Virtual Machine and run a new installation of YourKit in there - unless someone else knows a method to extend the evaluation period a little? I don't really need the tool except for showing the performance issues in the current JabRef version compared to version 3.8.2

@AEgit AEgit mentioned this issue Jun 23, 2019
Open
@tobiasdiez tobiasdiez added this to the v5.0 milestone Sep 27, 2019
@tobiasdiez tobiasdiez mentioned this issue Oct 28, 2019
Merged
6 tasks
@tobiasdiez
Copy link
Member

This should be fixed now. A few remaining performance issues are mentioned at #5071 (comment)

@AEgit
Copy link
Author

AEgit commented Nov 29, 2019

JabRef 5.0.0-dev--2019-11-28----ce0e8870c
Windows 10 10.0 amd64
Java 13.0.1

This issue, indeed, appears to be fixed in the current version of JabRef (the remaining issues can be found here: #5071 (comment)). Big shout out to @tobiasdiez for the continuous support!

Siedlerchr added a commit that referenced this issue Dec 11, 2019
@Siedlerchr
Squashed 'src/main/resources/csl-styles/' changes from f71cd32..49a1841
96c2e99 
49a1841 Update queen-margaret-university-harvard.csl (#4443)
f841560 Create uni-bern-institut-fur-musikwissenschaft-note.csl (#4401)
6ee503c Create harvard-cite-them-right-all-authors.csl (#4416)
3eceec4 Create harvard-queen-margaret-university.csl (#4441)
3c2299d JEMS.csl (#4436)
bb2f60f Create open-window.csl (#4430)
a908ab4 Nutrition Research Reviews (#4434)
208b693 Bump nokogiri from 1.10.3 to 1.10.4 (#4439)
122a3bb Create nmfs-west-coast-region-national-environmental-policy-act.csl (#4426)
3565fc2 Update tissue-engineering.csl (#4435)
981044e Create transposition.csl (#4375)
73ef6f3 Update tissue-engineering.csl (#4431)
7c4a4cf Update r-and-d-management.csl (#4432)
8628afb Create china-national-standard-gb-t-7714-2015-numeric.csl (#4065)
6ce148c removed spaces in harvard-gesellschaft-fur-bildung-und-forschung-in-europa.csl (#4422)
2af1c6f Create japanese-journal-of-applied-physics.csl (#4425)
fc16579 Add Presses universitaires de Strasbourg style (#4411)
01e58fb Correct spacing issues and documentation link (#4423)
d97cf3f Update geistes-und-kulturwissenschaften-heilmann.csl (#4421)
017947d Update ens-de-lyon-centre-d-ingenierie-documentaire.csl (#4413)
bffe57e Create instituto-de-pesquisas-energeticas-e-nucleares-sp.csl (#4368)
18686d1 Update and rename zeitschrift-fuer-digitale-geisteswissenschaften.csl… (#4410)
bd3a5f6 Final fix for journal article cit plus other updates - uniwersytet-im-adama-mickiewicza-w-poznaniu-wydzial-anglistyki (#4417)

git-subtree-dir: src/main/resources/csl-styles
git-subtree-split: 49a1841
@koppor koppor mentioned this issue Apr 14, 2020
Closed
@luizphilipe2 luizphilipe2 mentioned this issue Jun 26, 2021
Open
@koppor koppor moved this to Done in Prioritization Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Confirmed bugs or reports that are very likely to be bugs groups maintable type: performance
Projects
Prioritization
Archived in project
Milestone
v5.0
Development

No branches or pull requests
3 participants
@Siedlerchr @tobiasdiez @AEgit