Skip to content

chore(Analysis/InnerProductSpace): remove a hypothesis from LinearMap.IsSymmetric.card_filter_eigenvalues_eq#36106

Open
nielsvoss wants to merge 6 commits intoleanprover-community:masterfrom
RanadeLabBerkeley:spectrum_cleanup
Open

chore(Analysis/InnerProductSpace): remove a hypothesis from LinearMap.IsSymmetric.card_filter_eigenvalues_eq#36106
nielsvoss wants to merge 6 commits intoleanprover-community:masterfrom
RanadeLabBerkeley:spectrum_cleanup

Conversation

@nielsvoss
Copy link
Contributor

Use proof by cases to remove an unnecessary hypothesis from LinearMap.IsSymmetric.card_filter_eigenvalues_eq and the related private LinearMap.IsSymmetric.card_filter_unsortedEigenvalues_eq. This required proving a statement similar to LinearMap.IsSymmetric.hasEigenvalue_eigenvalues but for unsortedEigenvalues.


Open in Gitpod

@github-actions github-actions bot added the t-analysis Analysis (normed *, calculus) label Mar 4, 2026
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

PR summary 258952c8f9

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ hasEigenvalue_unsortedEigenvalues

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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant