[Merged by Bors] - chore(Analysis/Analytic): tag ofScalars_norm_eq_mul with simp

[vasnesterov]

Tag ofScalars_norm_eq_mul with simp and remove @[simp] from ofScalars_norm as it now can be proven using ofScalars_norm_eq_mul.

---

[github-actions]

PR summary 290b6cf8be

Import changes for modified files

No significant changes to the import graph

Import changes for all files

Files	Import difference

---

Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows

## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>

The doc-module for scripts/pr_summary/declarations_diff.sh contains some details about this script.

---

No changes to technical debt.

You can run this locally as

./scripts/reporting/technical-debt-metrics.sh pr_summary

• The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
• The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

[vasnesterov]

!bench

[leanprover-radar]

Benchmark results for a202151 against 290b6cf are in. There are no significant changes. @vasnesterov

• ✅ build//instructions: -4.6G (-0.00%)

Small changes (1🟥)

• 🟥 build/module/Mathlib.Tactic.Cases//instructions: +410.5M (+6.06%)

[themathqueen]

Without commenting on the simp tagging, this name of this and the other lemma below are wrong, they should be norm_ofScalars_....

[vasnesterov]

Should I rename it in this PR?

[themathqueen]

Probably a new PR so that this can stay on topic (i.e., adding @[simp])

[ocfnash]

It's not clear to be that ofScalars_norm_eq_mul should be simp. Could you give a bit of motivation?

[j-loreaux]

@ocfnash I think it's reasonable. Note that when the algebra satisfies NormOneClass, then ContinuousMultilinearMap.norm_mkPiAlgebraFin means the right-hand side will simplify to ‖c n‖, which seems like a thing we want.

@vasnesterov if Oliver doesn't complain in say, 24 hours, you can merge this.

bors d+

[mathlib-bors]

✌️ vasnesterov can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

[ocfnash]

Thanks, I'm convinced!

bors merge

[mathlib-bors]

Pull request successfully merged into master.

Build succeeded:

• ci (staging) / Lint style
• ci (staging) / Build
• ci (staging) / Post-CI job
• ci (staging) / Post-Build Step