Analysis stats are recomputed and reports are included in package regardless of selected option

This is very similar to https://community.sdl.com/product-groups/translationproductivity/f/90/t/9677
Looks like the new option to include existing reports in package, introduced back in Studio 2015 SR2 CU7, is improperly implemented :-\

Clearing the "Include reports in package" checkbox has no effect, Studio just ignores it and behaves like if it's set:

  • if the greyed-out radio button is set to "Recompute...", Studio recomputes the stats and then includes the single recomputed report in package
  • if the greyed-out radio button is set to "Include existing...", Studio includes all reports existing in the project folder(!) for the target language(s), including completely irrelevant ones like "Apply PerfectMatch", "Translation Count", "Pre-translate", etc.

The above behavior was observed in Studio 2017 CU5 (last pre-SR1 build), but I somehow doubt that any newer builds behave differently.

(The stats recomputing can be best seen with MANY files and LARGE TM... then the package creation takes considerably longer time than it should - and the progress stops in "Adding files" state for way longer than it should - plus the CPU load gets high for that time)

EDIT:The question is, what is actually implemented incorrectly - the behavior, or the GUI?!
Comparing to original functionality (only checkbox for recomputing or not recomputing the stats) it looks like the main problem is badly implemented GUI. And the inclusion of ALL reports instead of only the one relevant report is secondary problem...

Oh, and BTW, the developer also forgot to change the minimal height of the dialog (kept the original height before the new options were added), so the some options can be completely hidden!

Parents
  • Hi ,

    Thanks for the detail. I can confirm that the latest version doesn't behave any differently... I think this is a known issue already but I will report again just in case.

    I'd say the GUI is correct but the behaviour is not. Otherwise there would be no point in having the option at all.

    I can't say I noticed the height thing as I always see the right size. I can manually resize it though and this allows me to hide some of the text... this should not work this way.

    Paul Filkin | RWS Group

    ________________________
    Design your own training!

    You've done the courses and still need to go a little further, or still not clear? 
    Tell us what you need in our Community Solutions Hub

  • One reason I think the GUI is incorrect is the wording... in particular the plural form "Include reports..." and "Include existing reports...".
    The actual behavior seems to (almost) literally follow this wording - it includes ALL EXISTING reports (instead of only the relevant one).

    And the second - and main - reason is that recomputing the stats and including the reports to package are two separate things - user may simply want to recompute the stats (e.g. because that's the only chance to get stats for automatically splitted packages), but still NOT want to include the reports in the packages.
    Thus the current GUI is incorrect - it allows to recompute stats only if they are also included in the packages.

    In other words, user should not be forced to include the reports to packages just because he wants to recompute the stats.
    And the GUI should reflect this fact, e.g. this way:

  • I'm not sure I agree with you here but I sent the link to support with some comments so they can review your comments and make their own minds up.

    I think the GUI is explaining how it is intended to work. I doubt your consideration of a user who just wants to recompute stats at this stage but not include the reports was even considered.

    Paul Filkin | RWS Group

    ________________________
    Design your own training!

    You've done the courses and still need to go a little further, or still not clear? 
    Tell us what you need in our Community Solutions Hub

Reply Children