fix: Fix elementwise-apply if any input is AggregatedScalar

[reswqa]

This fixes #15602.

[reswqa]

IIUC, AggregatedScalar has the size equals to group len, It cannot do the fully element-wise apply. 🤔

[ritchie46]

I think it can if all acs are AggregatedScalar. I think any NotAggregated cannot combine with AggregatedList | AggregatedScalar (eg aggregated).

So if all are AggregatedScalar and the groups are equal we could still go int he elementwise, I think

[reswqa]

So if all are AggregatedScalar and the groups are equal we could still go int he elementwise, I think

Yes, you're right. I just think this All AggregatedScalar condition could be relaxed, AggregatedScalar and Literal can also go to the element-wise branch, right?

[codspeed-hq]

CodSpeed Performance Report

Merging #15606 will not alter performance

_{Comparing reswqa:agg_scalar_elementwise (df668e7) with main (23791bd)}

Summary

✅ 22 untouched benchmarks

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.19%. Comparing base (44f1097) to head (df668e7).
Report is 27 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #15606      +/-   ##
==========================================
+ Coverage   81.16%   81.19%   +0.02%     
==========================================
  Files        1367     1370       +3     
  Lines      175307   175639     +332     
  Branches     2527     2530       +3     
==========================================
+ Hits       142296   142606     +310     
- Misses      32534    32559      +25     
+ Partials      477      474       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ritchie46]

Yeap, good one!