bugGNU TeXmacs - Bugs: bug #42455, TeXmacs is slow

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #42455: TeXmacs is slow

Submitter:  Victor Porton <porton>
Submitted:  Wed 28 May 2014 09:27:12 PM UTC
 
Category:  Editor Priority:  7 - High
Item Group:  Error Status:  Confirmed
Privacy:  Public Assigned to:  vdhoeven
Originator Name:  Open/Closed:  Open
Release:  None Release: 
Fixed Release:  None Fixed Release: 
Keywords: 

( Jump to the original submission )

Thu 03 Jul 2014 07:17:26 AM UTC, comment #8: 

Youjun confirmed the performance improvement with the patch "revert_2655.diff".

As far as rev 8087 is concerned, is seems that it take more time to compute the rendering because more things are done : the typesetter recompute the document until references are defined (when possible). This is made in order to compute forward references properly for the first time. In theory, this should not impede computation for "well defined" documents.

Francois Poulain <fpoulain>
Project Member
Wed 02 Jul 2014 02:11:41 PM UTC, comment #7: 

Here is Youjun's document which highlight r2655's regression.

To see the regression, go in formula (1) and switch it to inline equation. The switch outline->inline makes the editor busy after rev 2655.

Francois Poulain <fpoulain>
Project Member
Wed 02 Jul 2014 12:20:18 PM UTC, comment #6: 

Fyi, I made tests with the document given by Youjun Hu, in the following email:

From: Youjun Hu <yjunhu@gmail.com>
To: Fran├žois Poulain <fpoulain@metrodore.fr>
Cc: "texmacs-users@texmacs.org" <texmacs-users@texmacs.org>
Subject: Re: [TeXmacs] Qt_version_of_TeXmacs-1.0.7.21_is_slow
Date: Tue, 1 Jul 2014 09:02:16 +0800

Francois Poulain <fpoulain>
Project Member
Wed 02 Jul 2014 12:16:41 PM UTC, comment #5: 

I spent many hours for searching another performance regression; and it was rewarding.

The rev r2655 introduced this :

--- a/src/Typeset/Bridge/bridge_document.cpp
+++ b/src/Typeset/Bridge/bridge_document.cpp
@@ -52,7 +52,7 @@ bridge_document_rep::initialize () {

 void
 bridge_document_rep::initialize_acc () {
-  if (ttt->paper) acc= bridge ();
+  if (true || ttt->paper) acc= bridge ();
   else acc= bridge_docrange (ttt, st, ip, brs, 0, N(st), true);
 }

I have no sounding idea of what this should make, but it clearly add a big overcomputing effort when removing lines is concerned.

Here is a patch for reverting it.

(file #31661)

Francois Poulain <fpoulain>
Project Member
Tue 01 Jul 2014 08:27:41 AM UTC, comment #4: 

Here is a patch for reverting r8087.

(file #31649)

Francois Poulain <fpoulain>
Project Member
Mon 30 Jun 2014 09:10:06 AM UTC, comment #3: 

I meant processing big document, not bug. :)

Francois Poulain <fpoulain>
Project Member
Fri 27 Jun 2014 01:58:22 PM UTC, comment #2: 

I made some tests on big a document on different versions of TeXmacs since 2012.

The performances are stable, divided in two periods. In fact, the rev 8087 has introduced a big overhead when processing bug documents. It needed (on my example) 16s to load the document before the commit, and 26s after.

Joris, I let you checking it.

Francois Poulain <fpoulain>
Project Member
Fri 30 May 2014 08:10:01 AM UTC, comment #1: 

Although improvements may be done, you are probably interested in splitting large documents. See e.g. http://www.texmacs.org/tmweb/manual/webman-links.en.html#auto-28 for help.

Francois Poulain <fpoulain>
Project Member
Wed 28 May 2014 09:27:12 PM UTC, original submission:  

TeXmacs 1.0.7.18 in a Debian Linux i7 machine is rather slow editing a 16979379 bytes (246 pages) book.

I'd like if the next version would be faster editing big files.

Victor Porton <porton>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #31666:  test_bug_42455.tm added by fpoulain (339KiB - text/texmacs)
file #31661:  revert_2655.diff added by fpoulain (478B - text/x-diff)
file #31649:  revert_8087.diff added by fpoulain (6KiB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mdbenito (Updated the item)
  • -email is unavailable- added by vdhoeven (Updated the item)
  • -email is unavailable- added by porton (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

     

     

    Follow 10 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-12-06 vdhoeven Priority6 - Bit higher 7 - High
    2015-03-22 mdbenito Priority5 - Normal 6 - Bit higher
    2014-07-02 fpoulain Attached File- Added test_bug_42455.tm, #31666
    2014-07-02 vdhoeven CategoryNone Editor
        Item GroupNone Error
    2014-07-02 fpoulain Attached File- Added revert_2655.diff, #31661
    2014-07-01 fpoulain Attached File- Added revert_8087.diff, #31649
    2014-06-27 fpoulain StatusNone Confirmed
        Assigned toNone vdhoeven
        Release1.0.7.18

    Back to the top


    Powered by Savane 3.9