ESC meeting minutes: 2022-11-10

Miklos Vajna vmiklos at collabora.com
Thu Nov 10 15:32:30 UTC 2022


* Present:
     + Xisco, Stephane, Stephan, Michael S, Ilmari, Heiko, Caolan, Miklos, Olivier, Hossein, Eike, Gabriel

* Completed Action Items:
     + ignore enhancement severity for the “QA update” section (Xisco)

* Pending Action Items:
     + create the Jenkins job for macOS arm64 (Cloph)
     + document how to get the Jenkins stats (Xisco)

* Release Engineering update (Xisco)
     + 7.4 status: 7.4.3 rc2 in 1 week
     + 7.5 release plan: alpha soon, Cloph will clarify

* Documentation (Olivier)
     + Helpcontents2
         + Updates and fixes (B. Buzea, ohallot, R. Lima, buovjaga, A. Romedenne, J. Nabet)
     + Guides
         + Progressing review process (S. Masonsmith, K. Kriek)

     + Bugzilla Documentation statistics
         237(237) bugs open
     + Updates:
         BZ changes   1 week   1 month   3 months   12 months
            created     11(8)    36(2)      73(-5)     304(0)
          commented     25(20)   68(9)     184(8)     1205(-7)
           resolved      6(6)    16(3)      36(-1)     196(2)
     + top 10 contributors:
           Bogdan B made 36 changes in 1 month, and 37 changes in 1 year
           Rafael Lima made 15 changes in 1 month, and 230 changes in 1 year
           Roman Kuznetsov made 14 changes in 1 month, and 45 changes in 1 year
           Olivier Hallot made 14 changes in 1 month, and 452 changes in 1 year
           Heiko Tietze made 10 changes in 1 month, and 116 changes in 1 year
           Srebotnjak, Martin made 7 changes in 1 month, and 12 changes in 1 year
           *UNKNOWN* made 7 changes in 1 month, and 7 changes in 1 year
           Ilmari Lauhakangas made 6 changes in 1 month, and 90 changes in 1 year
           Kaganski, Mike made 6 changes in 1 month, and 121 changes in 1 year
           Stéphane Guillou (stragu) made 6 changes in 1 month, and 7 changes in 1 year

* UX Update (Heiko)
     + Bugzilla (topicUI) statistics
         275(275) (topicUI) bugs open, 86(86) (needsUXEval) needs to be evaluated by the UXteam
     + Updates:
         BZ changes   1 week    1 month    3 months   12 months
              added      13(1)     26(-24)    52(-20)     75(-19)
          commented     105(70)   226(-46)   641(-37)   2359(-19)
            removed       2(2)      2(2)      10(1)       35(1)
           resolved      10(9)     18(-3)     80(7)      322(5)
     + top 10 contributors:
           Heiko Tietze made 90 changes in 1 month, and 1544 changes in 1 year
           Eyal Rozenberg made 84 changes in 1 month, and 247 changes in 1 year
           V Stuart Foote made 71 changes in 1 month, and 71 changes in 1 year
           Rafael Lima made 45 changes in 1 month, and 226 changes in 1 year
           Roman Kuznetsov made 41 changes in 1 month, and 223 changes in 1 year
           Dieter made 35 changes in 1 month, and 233 changes in 1 year
           Cor Nouws made 17 changes in 1 month, and 66 changes in 1 year
           Bogdan B made 15 changes in 1 month, and 40 changes in 1 year
           Henschel, Regina made 13 changes in 1 month, and 51 changes in 1 year
           Kaganski, Mike made 11 changes in 1 month, and 147 changes in 1 year
     + [Bug 151951] navigator >> send outline to clipboard
     + [Bug 146246] Residual UI toggle from old Toolbox based Navigator control,
            but it is non-functional in the present "by element" Navigator
     + [Bug 151926] Preview thumbnails should be able to have their size changed,
            and should be stored as an image file.
     + [Bug 151937] Make the Erase command accessible via the toolbar
  -> + [Bug 151914] Allow to add / edit words with tags in custom dictionaries
     + [Bug 151918] LO Default Language setting must only apply to a document
            on creation
     + [Bug 151906] If documents have a "document language", allow viewing and
            setting it
     + [Bug 149305] Increase/Decrease indent buttons should be part of the Text
            Formatting toolbar
     + [Bug 146950] clarify difference between "screen" and "on-screen show" in
            slide's paper format
  -> + [Bug 151880] CALC inserting new rows into conditional formatted column
            creates orphan and non-contiguous cell references

  -> + Better UX to insert and manage playing an audio stream/soundtrack in
       background for entire ODF presentation
       + https://bugs.documentfoundation.org/show_bug.cgi?id=151394
       + good as GSoC project
  -> + Toolbar button for enclosing text in () brackets
       + https://bugs.documentfoundation.org/show_bug.cgi?id=151710
       + possible to realize as autocomplete?

* Crash Testing (Caolan)
     + 22(+0) import failure, 0(+0) export failures
       + stable, not getting worse
     + 2 coverity issues
     + 6 ossfuzz issues
       - 1 leak -> https://bugs.documentfoundation.org/show_bug.cgi?id=151930 diagram models

* Crash Reporting (Xisco)
    + https://crashreport.libreoffice.org/stats/version/7.3.6.2
          + (-1) 796 797 802 695 605 523 391 313 139 0
    + https://crashreport.libreoffice.org/stats/version/7.3.7.2
          + (+217) 217 0
    + https://crashreport.libreoffice.org/stats/version/7.4.1.2
          + (+32) 980 948 1288 1676 1942 1653 1202 783 0
    + https://crashreport.libreoffice.org/stats/version/7.4.2.3
          + (+100) 1732 1632 1253 650 0

    + Symbols in 7.3.7.2 seems to be missing, perhaps they were not uploaded
      + will check with Cloph next week

    + Blind fix for osl_closeFile

        + Thanks to Dr. David Alan Gilbert


* Easyhack update / mentoring (Hossein)
           committer...   1 week     1 month     3 months    12 months
                   open      86(42)     128(15)     144(19)      148(19)
                reviews     358(110)   1100(-18)   2854(-66)   11782(-68)
                 merged     266(-11)   1111(45)    3564(-10)   14840(-51)
              abandoned      11(-8)      60(2)      173(0)       843(-1)
            own commits     203(-13)    873(56)    2741(0)     11415(-33)
         review commits      84(41)     307(47)     720(58)     3032(-27)
         contributor...   1 week     1 month     3 months   12 months
                   open      52(-15)    224(-14)    260(-9)     278(-9)
                reviews     826(132)   2734(138)   7924(-6)   32622(-60)
                 merged      34(27)      83(24)     230(15)    1089(-16)
              abandoned      10(5)       28(4)       66(4)      379(7)
            own commits      50(39)     126(40)     259(40)    1023(-27)
         review commits       0(0)        0(0)        0(0)        0(0)
     + easyHack statistics:
        needsDevEval 9(9)   needsUXEval 1(1)   cleanup_comments 307(307)
        total 369(369)   assigned 19(19)   open 320(320)
     + top 10 contributors:
           Bogdan B made 13 patches in 1 month, and 13 patches in 1 year
           Armin Le Grand (allotropia) made 12 patches in 1 month, and 25 patches in 1 year
           Kevin Suo made 8 patches in 1 month, and 22 patches in 1 year
           Paris Oplopoios made 8 patches in 1 month, and 8 patches in 1 year
             made 5 patches in 1 month, and 18 patches in 1 year
           Omkar Acharekar  made 5 patches in 1 month, and 5 patches in 1 year
           Srebotnjak, Martin made 4 patches in 1 month, and 22 patches in 1 year
           Paris Oplopoios made 4 patches in 1 month, and 79 patches in 1 year
           Javier Salamanca made 3 patches in 1 month, and 7 patches in 1 year
           Calvince Otieno made 3 patches in 1 month, and 3 patches in 1 year
     + top 10 reviewers:
           Vajna, Miklos made 88 review comments in 1 month, and 544 in 1 year
           Vajngerl, Tomaž made 86 review comments in 1 month, and 398 in 1 year
           McNamara, Caolán made 82 review comments in 1 month, and 804 in 1 year
           Michael Stahl made 82 review comments in 1 month, and 1070 in 1 year
           Nabet, Julien made 72 review comments in 1 month, and 774 in 1 year
           Adolfo Jayme Barrientos made 66 review comments in 1 month, and 1052 in 1 year
           Xisco Fauli made 66 review comments in 1 month, and 876 in 1 year
           Ilmari Lauhakangas made 56 review comments in 1 month, and 232 in 1 year
           Hossein made 46 review comments in 1 month, and 258 in 1 year
           Thorsten Behrens made 44 review comments in 1 month, and 844 in 1 year
     + Patches automatically abandoned:
         Added the executable for running benchmark tests ( Akshit Kushwaha )
             - Restored
             - https://gerrit.libreoffice.org/#/c/122879
         benchmark: Drawing wavelines ( Akshit Kushwaha )
             - Restored
             - https://gerrit.libreoffice.org/#/c/123045
         benchmark: Drawing Grid ( Akshit Kushwaha )
             - Restored
             - https://gerrit.libreoffice.org/#/c/123333
         benchmark: Drawing Bitmap ( Akshit Kushwaha )
             - Restored
             - https://gerrit.libreoffice.org/#/c/123454
         benchmark: Draw Gradient ( Akshit Kushwaha )
             - Restored
             - https://gerrit.libreoffice.org/#/c/123900
         tdf#149256 Ship LibreOfficeKit headers in packages ( Hossein )
             - Restored
             - https://gerrit.libreoffice.org/#/c/134863
         Add string C++ SDK example ( Hossein )
             - Restored
             - https://gerrit.libreoffice.org/#/c/134920
     + big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
           Ximena Alcaman
           Amarjargal Gundjalam
           Bogdan B
           Emanuele Goldoni
           Henner Drewes
           Nalini Dash
           Sahil Butola

* Mentoring
   * Statistics
     * 34 merged commits from the contributors
     * 7 new contributors
   * Drafted a new EasyHack
     * Drop LibreOffice dependency on boost libraries
       https://wiki.documentfoundation.org/User:Hossein/Blog/boost
     * got lots of feedback on IRC
     * maybe it's just the wording (Ilmari)
       + could the goal be: "minimize" or "only use it when it's needed"
       + saw some places where there is C++17 replacement already for boost functionality (Hossein)
       + could make the easy hack very specific
       + think there are many specific cases
       + though was that these specific cases were already cleaned up (Stephan)
         + if we have any, then make sense to have individual easy hacks with explicitly spelling out what specific functionality to replace
       + will reword according tho the above input (Hossein)
       + will be more interesting to group not by module, but group by boost functionality (Stephan)
       + gnu radio did build a functionality -> replacement table (Hossein)
         https://github.com/gnuradio/gnuradio/issues/5891
         + can focus on code use / include, ignore linking
         + will research this
     * Ilmari's research from June 2022: https://lists.freedesktop.org/archives/libreoffice/2022-June/089060.html

* Commit Access

* Developer Certification (Stephan/Kendy/Thorsten/László)
   + resting since: 8 weeks (limit: 20 weeks)

* Jenkins / CI update (Cloph)
   + there were occasional cases on Windows, bots misbehaving (Stephan)
     + just took them offline + pinged Cloph
     + good to have a backup for Cloph for such cleanups
AI: document how to do this & learn it from Cloph (Xisco)

* Remove the feature for positioning of mouse on "Default Button" (Ilmari)
   + Proposal came up in a bug: https://bugs.documentfoundation.org/show_bug.cgi?id=147469#c2
   + supporting the removal (Caolan)
   + perhaps accessibility would be a use case (Heiko)
     + no idea about the "centering" use-case
     + tried with "gen", it works with an actual X server
     + available as an OS setting on Windows: https://www.guidingtech.com/13050/automatically-move-mouse-pointer-default-button-dialog-box/ (Stephane)
   + suggest: remove the UI, and only remove the functionality later (Heiko)
AI: easyhackify this (Heiko)

* What’s cooking (Miklos)
     + testcase refactors (Xisco)
       + now also sw/ tests inherit from UnoApiTest
       + sc/ still needs doing
     + a new riscv64 bridge (Stephan)
       + author claimed that the UNO bridge tests work
       + still need to see if make check passes as well

  * Welcome Stéphane! (stragu)
     + joined TDF as a QA person on Monday

* QA update (Xisco)
     + UNCONFIRMED: 1827 (-15)
         + enhancements: 304  (-6)
         + needsUXEval: 34 (-16)
         + haveBackTrace: 7 (+0)
         + needsDevAdvice: 28 (+1)
         + documentation:  14 (+1)
         + android:  3 (+0)
         + iOS:  0 (+0)
         + Online:  11 (-3)


     + Most pressing bugs:
      New:

     None


     + New high severity bugs of the week:

     None


     + is it a good idea to start with bibisect requests, starting with the most recent ones? (Stephane)
       + yes (Xisco)
       + in many cases it's not possible to bisect (Ilmari)


* QA stats
     + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
       +110  -11 (-126) overall)
       many thanks to the top bug squashers:
        QA Administrators       14
        Heiko Tietze            13
        Caolán McNamara         9
        Rafael Lima             9
        Alex Thurgood           7
        Julien Nabet            7
        Aron Budea              6
        BogdanB                 6
        Dieter                  5
        Buovjaga                3

     + top 10 bugs reporters:
        Robert Großkopf         5
        Telesto                 4
        Eyal Rozenberg          3
        TorrAB                  3
        Miklos Vajna            2
        Muhammad Wajahat Asif   2
        Regina Henschel         2
        Mike Kaganski           1
        Orwel                   1
        Ulrich Windl            1

     + top 10 bugs fixers:
        McNamara, Caolán        9
        Bogdan B                6
        Rafael Lima             4
        Adolfo Jayme Barrientos 2
        Henschel, Regina        2
        Nabet, Julien           2
        Samuel Mehrbrodt        2
                                1
        Aron Budea              1
        Bee                     1

     + top 10 bugs confirmers:
        Heiko Tietze            18
        Dieter                  14
        Roman Kuznetsov         8
        m.a.riosv               7
        McNamara, Caolán        6
        Nabet, Julien           6
        Alex Thurgood           4
        Raal                    4
        Rafael Lima             4
        *UNKNOWN*               2


* Highest-Priority bugs (aka "MABs"):
    + http://bit.ly/2lJAAJI
      6.0 : 1/53 -     1 %  (+0)
      5.4 : 0/41 -     0 %  (+0)
      5.3 : 0/54 -     0 %  (+0)
      5.2 : 0/43 -     0 %  (+0)
      5.1 : 0/35 -     0 %  (+0)
      5.0 : 0/64 -     0 %  (+0)
      4.4 : 0/76 -     0 %  (+0)
      4.3 : 0/73 -     0 %  (+0)
      4.2 : 2/134 -    1 %  (+0)
      4.1 : 2/85 -     2 %  (+0)
      4.0 : 1/83 -     1 %  (+0)
      old : 1/246 -    0 %  (+0)


* Bisected bugs open: keyword 'bisected'
    + more accurate - down to a single commit.
    + http://bit.ly/2dyIfDy
    +

      done by:
         Raal                   6
         Terrence Enger         1


* Bibisected bugs open: keyword 'bibisected'
    + http://bit.ly/2cSCXlS
    +

      done by:
         Raal                   6
         Terrence Enger         1


* all bugs tagged with 'regression'
    + 1176(+5) bugs open of 12111(+18) total 8(+0) high prio.

      done by:
         Robert Großkopf        5
         m.a.riosv              3
         Alex Thurgood          2
         Bielefeld, Rainer      2
         Raal                   2
         Telesto                2
         Dieter                 1


* ~Component   count net * high severity regressions
    + http://bit.ly/1HWHb3E
      Writer        - 5(+0)
      Calc          - 1(+0)
      Draw          - 1(+0)
      LibreOffice   - 1(+0)

    by OS:
      All           - 7(+0)
      Linux         - 0(+0)
      Mac OS X      - 0(+0)
      Windows       - 1(+0)


* ~Component   count net * all regressions
    + http://bit.ly/1BUdI8i
      Writer: other            - 422(+4)
      Calc                     - 195(+3)
      Impress                  - 117(-1)
      Writer: docx filter      - 57(+0)
      Crashes                  - 53(+2)
      LibreOffice              - 44(+0)
      Draw                     - 41(+1)
      Base                     - 39(+2)
      UI                       - 36(-1)
      Writer: perf             - 36(+0)
      Borders                  - 32(+0)
      Writer: other filter     - 31(+0)
      Chart                    - 24(+0)
      RTL                      - 23(+0)
      Writer: doc filter       - 19(+0)
      BASIC                    - 17(+0)
      Printing and PDF export  - 15(+0)
      filters and storage      - 13(+0)
      graphics stack           - 13(+0)
      Formula Editor           - 3(+0)
      sdk                      - 2(+0)
      framework                - 1(+0)


More information about the LibreOffice mailing list