Protection in all_invariant_masses computation

[atishelmanch]

Hello FCCAnalyses experts - when I implemented all_invariant_masses a while ago, I forgot to include a protection on the numbers of jets. This is problematic when running on events that do not have at least 2 jets - which can happen for example with the WW background even if you require exclusive clustering with njets = 4. See attached the error, stage 1 config file, and command run to produce the error:

error.log
cmd.log
ZH_Hadronic_stage1.py.txt

The error is fixed for me when adding the simple protection.

---

Files edited:

• analyzers/dataframe/src/JetConstituentsUtils.cc:
  • Added the protection.

---

By the way, I've encountered the same problem when importing this weaver example as by default it computes invariant mass assuming you have two jets. Given that this is just an example, not sure if the protection is necessary here.

[kjvbrt]

Hi @atishelmanch, the fix looks OK to me :)

Can you add also a little documentation about the analyzer in the form similar to this example?

/**
 * @brief Use brief, otherwise the index won't have a brief explanation.
 *
 * Detailed explanation.
 */
typedef struct BoxStruct_struct {
  int a;    /**< Some documentation for the member BoxStruct#a. */
  int b;    /**< Some documentation for the member BoxStruct#b. */
  double c; /**< Etc. */
} BoxStruct;

[atishelmanch]

Hi @atishelmanch, the fix looks OK to me :)

Can you add also a little documentation about the analyzer in the form similar to this example?

/**
 * @brief Use brief, otherwise the index won't have a brief explanation.
 *
 * Detailed explanation.
 */
typedef struct BoxStruct_struct {
  int a;    /**< Some documentation for the member BoxStruct#a. */
  int b;    /**< Some documentation for the member BoxStruct#b. */
  double c; /**< Etc. */
} BoxStruct;

@kjvbrt - like this?

[atishelmanch]

Friendly ping @kjvbrt - 🙂 is this ready to merge?

[kjvbrt]

Hi @atishelmanch, Yes :)