ESC meeting minutes: 2022-02-17

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Present:
    + Cloph, Heiko, Kendy, Olivier, Caolan, Hossein, Michael W, Michael S, Stephan, Miklos, Xisco, Michael M, Eike, Gabriel

* Completed Action Items:
    + try to revert the wasm-related change that results in .component loading build failures and see if it really helps (Cloph)
      + done, thanks Noel

* Pending Action Items:
    + None

* Release Engineering update (Cloph)
    + 7.3 status: 7.3.1 rc2 in 1 week
      + rc1 was waiting for the AVX usage revert
      + rc1 tag was force-pushed, as the version bump was missing
    + 7.2 status: 7.2.6 rc2 in 2 weeks
    + Remotes: Android, iOS
    + Android viewer

* Documentation (Olivier)
    + New Help:
        + fixes (buovjaga)
        + improvements to come (R. Lima)

    + Helpcontents2
        + Maintenance (Johnny_M, A. Gelmini)
        + Typos after translation round (S. Horacek)
        + Updates (S. Chaiklin, Olivier, A. romedenne, R. Lima, A Barrientos)

    + Guides
        + Update on Base Guide
            + opportunity to review Base (Vasudev, flywire)
            + Coordination by ohallot
            + time based release of the guide.
        + Getting started Guide
            + final revisions (K. Kriek, P. Schofield, J Weber)

     + Google Season of Doc
        + Open for ideas, please speak up !

    + Bugzilla Documentation statistics
        241(241) bugs open
    + Updates:
        BZ changes   1 week   1 month    3 months   12 months
           created     15(2)     51(-11)   125(10)     333(-1)
         commented     34(1)    156(-25)   454(-2)    1652(-17)
          resolved      4(1)     17(-5)     59(4)      201(0)
    + top 10 contributors:
          Olivier Hallot made 63 changes in 1 month, and 579 changes in 1 year
          Nabet, Julien made 23 changes in 1 month, and 62 changes in 1 year
          Heiko Tietze made 22 changes in 1 month, and 95 changes in 1 year
          Rafael Lima made 22 changes in 1 month, and 370 changes in 1 year
          Ilmari Lauhakangas made 19 changes in 1 month, and 120 changes in 1 year
          Kaganski, Mike made 16 changes in 1 month, and 99 changes in 1 year
          Michael Warner made 9 changes in 1 month, and 10 changes in 1 year
          Nucleos made 8 changes in 1 month, and 8 changes in 1 year
          Xisco Fauli made 8 changes in 1 month, and 88 changes in 1 year
          Alain Romedenne made 7 changes in 1 month, and 34 changes in 1 year

* UX Update (Heiko)
    + Bugzilla (topicUI) statistics
        259(259) (topicUI) bugs open, 105(105) (needsUXEval) needs to be evaluated by the UXteam
    + Updates:
        BZ changes   1 week   1 month   3 months   12 months
             added      2(-1)     8(-7)    13(-8)      61(-8)
         commented     55(4)    233(1)    649(-25)   2810(-6)
           removed      3(2)      7(2)     12(2)       35(-1)
          resolved     14(4)     48(4)    105(3)      403(0)
    + top 10 contributors:
          Heiko Tietze made 250 changes in 1 month, and 1938 changes in 1 year
          Seth Chaiklin made 35 changes in 1 month, and 136 changes in 1 year
          Foote, V Stuart made 22 changes in 1 month, and 487 changes in 1 year
          Telesto made 14 changes in 1 month, and 293 changes in 1 year
          Timur made 12 changes in 1 month, and 113 changes in 1 year
          Dieter made 11 changes in 1 month, and 247 changes in 1 year
          Ilmari Lauhakangas made 11 changes in 1 month, and 299 changes in 1 year
          Albrecht Müller made 10 changes in 1 month, and 10 changes in 1 year
          *UNKNOWN* made 10 changes in 1 month, and 10 changes in 1 year
          Kainz, Andreas made 10 changes in 1 month, and 32 changes in 1 year

    + 6 new tickets with needsUXEval Feb/11-17
      + [Bug 147473] Rearrange items the Numbering type list on Customize tab
             in B&N dialog
      + [Bug 147444] Calc - interaction between filters
      + [Bug 147286] ‘Default Paragraph Style’ should be renamed ‘Basic’
      + [Bug 147373] UI: Default file format on export mess (Draw:GIF;
             Calc: WEBP: Writer:EPUB; Impress:PDF)
      + [Bug 147346] simplify gallery usage
      + [Bug 147344] Integrate Auto Text to gallery sidebar

* Crash Testing (Caolan)
    + 13(+0) import failure, 6(+0) export failures
      - a new calc limit related issue, now fixed
    + 4 coverity issues
      - still using the old coverity, will bring to zero again and try new coverity when website returns
    + 9 ossfuzz issues

* Crash Reporting (Xisco)
   + https://crashreport.libreoffice.org/stats/version/7.2.5.2
         + (+2) 1778 1776 1707 1860 1608 1246 691 0
   + https://crashreport.libreoffice.org/stats/version/7.3.0.3
         + (+124) 2384 2260 347 0

   + https://crashreport.libreoffice.org/stats/signature/_GLOBAL__sub_I_protectiondlg.cxx.lto_priv.2643

       - Should be fixed in 7.3.1.1 - Thanks to Lubos


    + https://crashreport.libreoffice.org/stats/signature/rtl::str::assign%3C_rtl_uString%3E(_rtl_uString%20*%20*,_rtl_uString%20*)

    - Happening in LibreOffice 7.2.x only

    - not yet clear what commit fixed it


* Mentoring/EasyHack update (Hossein)
          committer...   1 week      1 month     3 months     12 months
                  open      47(-38)      94(-32)    150(-19)      204(-15)
               reviews     348(-100)   1396(-16)   3498(-10)    12100(228)
                merged     271(-63)    1348(-3)    3948(-104)   14745(266)
             abandoned      16(2)        62(-3)     197(5)        688(16)
           own commits     203(-55)    1074(-6)    3065(-54)    12588(-78)
        review commits      75(-21)     345(19)     947(-39)     3951(-47)
        contributor...   1 week      1 month     3 months     12 months
                  open      16(-1)       28(-5)      75(-15)      181(-3)
               reviews     700(-250)   3126(-40)   8774(-170)   32692(544)
                merged      21(-12)     100(19)     267(-22)     1600(20)
             abandoned       4(-11)      26(3)       57(-1)       435(4)
           own commits      31(-15)     130(30)     381(-44)     1745(-21)
        review commits       0(0)         0(0)        0(0)          0(0)
    + easyHack statistics:
       needsDevEval 8(8)   needsUXEval 1(1)   cleanup_comments 289(289)
       total 340(340)   assigned 27(27)   open 284(284)
    + top 10 contributors:
          Johnny_M made 76 patches in 1 month, and 904 patches in 1 year
          Attila Bakos made 5 patches in 1 month, and 30 patches in 1 year
          Po-Yen Huang made 5 patches in 1 month, and 12 patches in 1 year
          Kevin Suo made 4 patches in 1 month, and 35 patches in 1 year
          Vaibhav Malik made 4 patches in 1 month, and 9 patches in 1 year
          Dhiraj Holden made 3 patches in 1 month, and 6 patches in 1 year
          Donkers, Winfried made 2 patches in 1 month, and 4 patches in 1 year
          Nagy Tibor made 2 patches in 1 month, and 34 patches in 1 year
          jwtiyar nariman made 2 patches in 1 month, and 2 patches in 1 year
          Attila Szűcs made 1 patches in 1 month, and 18 patches in 1 year
    + top 10 reviewers:
          Olivier Hallot made 126 review comments in 1 month, and 1306 in 1 year
          Michael Stahl made 124 review comments in 1 month, and 996 in 1 year
          Adolfo Jayme Barrientos made 116 review comments in 1 month, and 914 in 1 year
          Xisco Fauli made 92 review comments in 1 month, and 854 in 1 year
          Thorsten Behrens made 88 review comments in 1 month, and 694 in 1 year
          Kaganski, Mike made 86 review comments in 1 month, and 802 in 1 year
          Nabet, Julien made 84 review comments in 1 month, and 510 in 1 year
          Rathke, Eike made 52 review comments in 1 month, and 308 in 1 year
          Bergmann, Stephan made 50 review comments in 1 month, and 226 in 1 year
          Heiko Tietze made 48 review comments in 1 month, and 272 in 1 year
    + big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
          javier torres
          jwtiyar nariman
          Renwa Hiwa
          Sinduja Y
          Pragat Pandya
          Kunal Pawar
          + 2 of these are language-related (locale data, etc)
          + trying to steer them towards more complex tasks
    + draft blog post: adding new UNO command, WIP

* GSoC (Ilmari)
    + GSoC 2022 application is filed
    + https://developers.google.com/open-source/gsoc/timeline
    + next step: 7th March: accepted orgs are announced
    + just waiting for now (Thorsten)

* Commit Access
  + None

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

* Jenkins / CI update (Cloph)
    from:Thu Feb 10 14:56:35 2022
    master linux rel  jobs: 121 ok: 121 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master linux dbg  jobs: 107 ok:  85 ko:  22 fail ratio: 20.56 % break:   8 broken duration:15.53%
    master mac rel    jobs:  89 ok:  89 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master mac dbg    jobs:  93 ok:  93 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master win rel    jobs:  86 ok:  86 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master win dbg    jobs:  87 ok:  86 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master win64 dbg  jobs:  85 ok:  74 ko:  11 fail ratio: 12.94 % break:   1 broken duration: 9.08%
    master gerrit lin jobs: 439 ok: 348 ko:  69 fail ratio: 15.72% time for ok: mean:  19 median:  16
    master gerrit plg jobs: 380 ok: 240 ko: 110 fail ratio: 28.95% time for ok: mean:  42 median:  40
    master gerrit win jobs: 601 ok: 289 ko: 264 fail ratio: 43.93% time for ok: mean:  54 median:  52
    master gerrit mac jobs: 436 ok: 311 ko:  94 fail ratio: 21.56% time for ok: mean:  35 median:  29
    master android    jobs: 370 ok: 314 ko:  28 fail ratio:  7.57% time for ok: mean:  20 median:  14
    master gerrit all jobs: 421 ok: 207 ko: 157 fail ratio: 37.29% time for ok: mean:  95 median:  71
   + https://dev-www.libreoffice.org/tmp/gerrit_220217.html
   + tests that failed more than twice in last seven days
      3 CppunitTest_sw_layoutwriter                gerrit_linux_clang_dbgutil
      5 Killed by the kill-wrapper                gerrit_linux_clang_dbgutil
     35 CppunitTest_sc_macros_test                gerrit_windows (22 on a single bot)

   + was a good week in short
     + investigation with .component loading on Windows was not entirely successful, but improved cygwin load balancing -> 10-15 mins faster (Cloph)
     + with load limiting: https://frupic.frubar.net/shots/42843.png
     + only with parallelism: https://frupic.frubar.net/shots/42848.png

* l10n (Sophie)
  + removal of product name from extended tips -> was removed due to performance
    + affects l10n

* Various baseline bumps (Stephan)
  + Update on bumping macOS build baseline to Xcode 12.5 and macOS 11.0
    + still outstanding, one bot is not yet updated (Cloph)
  + Suggestion to bump Clang loplugin baseline to Clang 12.0.1
    + rolling it out in Jenkins
  + and bump Clang non-loplugin baseline on Linux and Android to Clang 8.0.1
    + turns out Android uses some custom clang, but "clang 8" seemt to work

* Naming for new environment variables (Miklos)
  + related to https://gerrit.libreoffice.org/c/core/+/129992 from Heiko
  + recently introduced SAL_NO_QUERYSAVE was renamed to NO_QUERYSAVE
  + existing prefixes are LO_ or '<named of module>_' (e.g. SAL_)
  + a bit concerned about no prefixes, easy to affect behavior of other software if you put that in some global config (e.g. bashrc)
  + list of env vars at https://wiki.documentfoundation.org/Development/Environment_variables
  + makes sense to have some prefix (Michael W)
  + no problem with having a prefix (Heiko)
    + don't like the mess with LIBO_, LO_, etc.
  + was pointing out just the "SAL_" prefix, not hinting no prefixes (Stephan)
  + fine with any prefix (Stephan)
  + SAL_ prefix usually means the "sal" part of VCL (Michael S)
  + don't like tons of new environment variables anyway (Stephan)
    + too many settings
    + increasing complexity
    + they are hard to document
    + the list of environment variables is not perfect
      + some of them are build-time names
      + some other are missing, when not going via getenv()
AI: add a note to the wiki page to have prefixes (to prevent collision) + think twice before adding a new one (Hossein)
AI: go back to have a prefix for NO_QUERYSAVE (Heiko)

      + https://gerrit.libreoffice.org/c/core/+/130026


* What’s cooking (Miklos)
  + Lubos is working towards non-experimental jumbo sheets in Calc
    + hopes to enable 16k cols as non-experimental on master perhaps next week
    + intention is to have more feedback, early

* QA update (Xisco)

    + UNCONFIRMED: 1688 (+19)
        + enhancements: 265  (-1)
        + needsUXEval: 2 (-7)
        + haveBackTrace: 6 (+1)
        + needsDevAdvice: 25 (+0)
        + documentation:  5 (+4)
        + android:  1 (+0)
        + iOS:  0 (+0)
        + Online:  10 (+0)


    + Most pressing bugs:
     New:

    None

     Old:
        + Slow typing with version 7.1 or above on Windows 11
            + https://bugs.documentfoundation.org/show_bug.cgi?id=145963
            + Lubos?
            + seems to be Windows 11-only

        + macOS: Can't save or open files using Finder dialog on Standard accounts on macOS 10.15 Catalina
           + https://bugs.documentfoundation.org/show_bug.cgi?id=128233
           + started to happen with a new macOS version

        + Text runs of RTL scripts (e.g. Arabic, Hebrew, Persian) from
          imported PDF are reversed, PDFIProcessor::mirrorString not behaving
            + https://bugs.documentfoundation.org/show_bug.cgi?id=104597
            + already listed here for 2 years, and many duplicates
    Fixed:
        + Crash in: _GLOBAL__sub_I_protectiondlg.cxx.lto_priv.2643 ( Calc crashes )
            + https://bugs.documentfoundation.org/show_bug.cgi?id=147135
            + Thanks to Lubos

    + New high severity bugs of the week:
        + Writer stops responding when a group of shapes is anchored to character
            + https://bugs.documentfoundation.org/show_bug.cgi?id=146597
            + Daniel?

* QA stats
    + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
      +154  +26 (-158) overall)
      many thanks to the top bug squashers:
       Heiko Tietze            15
       QA Administrators       14
       Justin L                11
       Luboš Luňák             10
       Xisco Faulí             9
       Buovjaga                8
       Gabor Kelemen (allotropia) 7
       Caolán McNamara         6
       Timur                   6
       Dieter                  5

    + top 10 bugs reporters:
       Telesto                 14
       Mike Kaganski           7
       Alex Thurgood           4
       Colin                   3
       Heiko Tietze            3
       BogdanB                 2
       Dreamquartz             2
       Gabor Kelemen (allotropia) 2
       Jens-D Doll             2
       Luke Kendall            2

    + top 10 bugs fixers:
       Luboš Luňák             7
       Heiko Tietze            5
       Xisco Fauli             5
       Justin Luth             4
       McNamara, Caolán        4
       Németh, László          4
       Andreas Heinisch        3
       Melenchuk, Vasily       2
       Michael Stahl           2
       Nabet, Julien           2

    + top 10 bugs confirmers:
       Raal                    14
       Dieter                  12
       m.a.riosv               12
       Heiko Tietze            7
       Xisco Fauli             6
       Ezinne Nnamani          4
       Telesto                 4
       *UNKNOWN*               3
       Aleksandra              3
       Alex Thurgood           3


* 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 : 1/36 -     2 %  (+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 : 3/85 -     3 %  (+0)
     4.0 : 2/83 -     2 %  (+0)
     old : 2/247 -    0 %  (+0)


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

     done by:
        Raal                   6
        Aron Budea             4
        Timur                  2
        Kelemen, Gabor         2
        Roman Kuznetsov        1
        Ilmari Lauhakangas     1
        Terrence Enger         1
        Xisco Fauli            1


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

     done by:
        Raal                   5
        Aron Budea             4
        Timur                  2
        Kelemen, Gabor         2
        Roman Kuznetsov        1
        Ilmari Lauhakangas     1
        Terrence Enger         1
        Xisco Fauli            1


* all bugs tagged with 'regression'
   + 1261(-19) bugs open of 11573(+11) total 11(+0) high prio.

     done by:
        Telesto                4
        Robert Großkopf        2
        Ilmari Lauhakangas     1
        Kelemen, Gabor         1
        m.a.riosv              1
        Németh, László         1
        Raal                   1
        Nabet, Julien          1
        Xisco Fauli            1


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

   by OS:
     All           - 9(+0)
     Linux         - 0(+0)
     Mac OS X      - 0(+0)
     Windows       - 2(+0)


* ~Component   count net * all regressions
   + http://bit.ly/1BUdI8i
     Writer: other            - 435(+1)
     Calc                     - 217(-5)
     Impress                  - 124(-6)
     Writer: docx filter      - 57(-1)
     LibreOffice              - 49(+0)
     Crashes                  - 48(+0)
     Draw                     - 45(+0)
     UI                       - 44(-1)
     Borders                  - 40(+0)
     Writer: perf             - 37(-1)
     Writer: other filter     - 35(+0)
     Base                     - 33(+0)
     Chart                    - 29(+0)
     RTL                      - 26(+0)
     Writer: doc filter       - 22(+0)
     Printing and PDF export  - 21(+0)
     graphics stack           - 20(+0)
     BASIC                    - 18(-1)
     filters and storage      - 16(+0)
     Formula Editor           - 2(+0)
     framework                - 1(+0)
     sdk                      - 1(+0)



[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux