Recreate 36964 #5

- Cherry-picked from sagemath#35095. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38071 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

…date `# needs`       - Cherry-picked from sagemath#35095. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38074 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

as another move towards removal of the "Algebra" class, using "Parent" framework instead. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [x] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. URL: sagemath#38078 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

This fixes all warnings for pep8 E302 in `algebras` and `categories` folders : ``` E302 expected 2 blank lines, found 0 or 1. ``` This was purely scripted using `autopep8`. ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38079 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

this is removing empty lines in the `libs` folder in `.pyx` files, for compliance with pep8 E302 and E305 ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38080 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

Fixes sagemath#37873 (comment) URL: sagemath#38081 Reported by: Antonio Rojas Reviewer(s):

as seen in https://github.com/sagemath/sage/actions/runs/9234699215/job/25408704331 #:~:text=bash%3A%20line%201%3A%20cd%3A%20./worktree%2Dimage%3A%20No%20su ch%20file%20or%20directory ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38082 Reported by: Kwankyu Lee Reviewer(s):

…l_doc'       - Cherry-picked from sagemath#37900 Includes style fixes from there as well. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38086 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

…ilt our own toolchain The GMP configure test should link against our own libstdc++ instead of the (too old) system libstdc++ Fixes sagemath#38085 URL: sagemath#38087 Reported by: Volker Braun Reviewer(s):

…repair Coverage.py upload       test-long (90 minutes) runs after a successful test-new (10 minutes). This is the critical path (100 minutes), longer than build-docs-pdf (85 minutes). We split it into multiple parallel jobs (33 minutes), merging the coverage files in a job afterwards (6 minutes). This also repairs the Coverage.py upload, which apparently got broken. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38090 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

``` ********************************************************************** File "src/sage/features/__init__.py", line 384, in sage.features.Feature.hide Failed example: len(list(graphs.fusenes(2))) # optional - benzene, needs sage.graphs Expected nothing Got: 1 ********************************************************************** 1 item had failures: 1 of 6 in sage.features.Feature.hide [146 tests, 1 failure, 3.74 s] ********************************************************************** ``` Comes from 1ade282 URL: sagemath#38091 Reported by: Volker Braun Reviewer(s):

### 📝 Checklist - [x] The title is concise and informative. - [x] I have updated the documentation and checked the documentation preview. URL: sagemath#38092 Reported by: gmou3 Reviewer(s): Matthias Köppe

Just adding a little comment in the doc on the link to Borel and Laplace formal transforms. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have updated the documentation and checked the documentation preview. URL: sagemath#38093 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

…t, remove deprecated global import       - The change to use `lazy_import` was cherry-picked from sagemath#37900 - The global import was deprecated in sagemath#32647 (2021) ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38095 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

All deprecated 2019-2022. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38099 Reported by: Matthias Köppe Reviewer(s): Frédéric Chapoton

Fixes sagemath#37996. We remove this function ```python def nitpick_patch_config(app): """ Patch the default config for nitpicky [...] ensure that nitpicky is not considered as a Sphinx environment variable but rather as a Sage environment variable. As a consequence, changing it doesn't force the recompilation of the entire documentation. """ app.config.values['nitpicky'] = (False, 'sage') app.config.values['nitpick_ignore'] = ([], 'sage') ``` from `src/sage_docbuild/conf.py`. The original purpose of the function seems to change `"env"` to `"sage"` for `nitpicky` config variable. But the recent Sphinx is setting `""` (empty string) instead of `"env"` by default. See https://github.com/sphinx-doc/sphinx/blob/48cbb43e28efe82b198137042811e0 ade3599ae7/sphinx/config.py#L250C1-L250C41 - Hence the function is already no-op. But this old and obsolete function is the cause of sagemath#37996. Hence we simply remove it. - To prepare for Sphinx 7.3.7, we also remove some code which seems to have become obsolete when `sage -clone` was removed long time ago. - Made some docstring up-to-date, to test incremental doc build. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38100 Reported by: Kwankyu Lee Reviewer(s):

- Cherry-picked from sagemath#35095. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38104 Reported by: Matthias Köppe Reviewer(s): David Coudert, Kwankyu Lee, Matthias Köppe

Fixes sagemath#37979 by handling passing along the index set for the special cased rank 1 Cartan matrix. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [x] I have created tests covering the changes. - [x] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38105 Reported by: Travis Scrimshaw Reviewer(s): Frédéric Chapoton

…, `database_knotinfo`, `matroid_database`       Using `sage --package update-latest ... --commit`. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38110 Reported by: Matthias Köppe Reviewer(s): gmou3, Kwankyu Lee

The change made in sagemath#37099 to accommodate the new design of actions/{up,down}load-artifact@v4 broke the upload of macOS wheels to PyPI (see https://github.com/sagemath/sage/actions/runs/9234698750/job/2 5408786962#step:8:23); this was masked so far because the build of macOS wheels was broken until sagemath#36525. Here we revert to using a separate job that uploads all platform wheels. This uses the new keywords `pattern` and `merge-multiple` of actions/download-artifact@v4 Example run: https://github.com/mkoeppe/sage/actions/runs/9291646178/job/25572343853 ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38111 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

This is a bug in sagemath#37589. I made other small edits. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38121 Reported by: Kwankyu Lee Reviewer(s):

…f is_NumberFieldOrder

…erFieldFractionalIdeal_rel

…FFIX=" "CLASS='ToricLattice';SUFFIX=_generic"; do eval "$a"; git --no-pager grep -l -E 'is_('$CLASS')' src/sage | xargs sed -E -i.bak 's/import is_('$CLASS')/import \1'$SUFFIX'/;/^ +/s/is_('$CLASS')[(]([^()]*([(][^()]*[)])?[^()]*)[)]/isinstance(\2, \1'$SUFFIX')/g'; done

Bumps [dev-hanz-ops/install-gh-cli-action](https://github.com/dev-hanz-ops/install-gh-cli-action) from 0.1.0 to 0.2.0. - [Release notes](https://github.com/dev-hanz-ops/install-gh-cli-action/releases) - [Commits](dev-hanz-ops/install-gh-cli-action@v0.1.0...v0.2.0) --- updated-dependencies: - dependency-name: dev-hanz-ops/install-gh-cli-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>

…ies_distribution_directives_2024

…cation

…ainComplexMorphism|SimplicialComplexMorphism';SUFFIX=" ; do eval "$a"; git --no-pager grep -l -E 'is_('$CLASS')' src/sage | xargs sed -E -i.bak 's/import is_('$CLASS')/import \1'$SUFFIX'/;/^ +/s/is_('$CLASS')[(]([^()]*([(][^()]*[)])?[^()]*)[)]/isinstance(\2, \1'$SUFFIX')/g'; done

…ep -l -E 'is_('$CLASS')' src/sage | xargs sed -E -i.bak 's/import is_('$CLASS')/import \1'$SUFFIX'/;/^ +/s/is_('$CLASS')[(]([^()]*([(][^()]*[)])?[^()]*)[)]/isinstance(\2, \1'$SUFFIX')/g'; done

…ients): Fix for submodules

…ients

…ase ring

…ng workaround, delegate to .element()

…eModule

…multiplication

…nomial_coefficients to raise NotImplementedError

…ling tests

…ients): Use dict only when coordinate ring matches

…m global bindings

…36676+38088

…ot is Co-authored-by: Travis Scrimshaw <[email protected]>

Co-authored-by: Matthias Köppe <[email protected]>

…el condition

…-linux-incremental.yml)

…container if nothing to be tested

Deprecate `in/dependent_r_sets`

…flow warning

…of the default platform (ubuntu-jammy)

…dora-38 by latest

…t from build.yml, use ptest-nodoc to avoid duplicate doc-html build

…modules from sage.modules       - Rebased version of sagemath#34455 - Part of sagemath#30309 Fixes sagemath#34455 ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies    URL: sagemath#37526 Reported by: Matthias Köppe Reviewer(s): Matthias Köppe, Travis Scrimshaw

….0 to 0.2.0 Bumps [dev-hanz-ops/install-gh-cli-action](https://github.com/dev-hanz- ops/install-gh-cli-action) from 0.1.0 to 0.2.0. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/dev-hanz-ops/install-gh- cli-action/releases">dev-hanz-ops/install-gh-cli-action's releases</a>.</em></p> <blockquote> <h2>v0.2.0 - update to node20</h2> <ul> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/8fff9050dae2d81b38f94500d8b74ad1d1d47410">https://github.com/dev-hanz- ops/install-gh-cli- action/commit/8fff9050dae2d81b38f94500d8b74ad1d1d47410</a> (update to node20)</li> </ul> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/8fff9050dae2d81b38f94500d8b74ad1d1d47410"><code>8fff905</code></a> update to node 20</li> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/67bfd0539397d168c77ba70eac2647fef7f1f020"><code>67bfd05</code></a> README: only linux amd64</li> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/44745680363a7ba976f907c6d1b697a9fc311cf7"><code>4474568</code></a> Update readme with working example (<a href="https://redirect.github.com/dev-hanz-ops/install-gh-cli- action/issues/3">sagemath#3</a>)</li> <li>See full diff in <a href="https://github.com/dev-hanz-ops/install- gh-cli-action/compare/v0.1.0...v0.2.0">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot- badges.githubapp.com/badges/compatibility_score?dependency-name=dev- hanz-ops/install-gh-cli-action&package-manager=github_actions&previous- version=0.1.0&new- version=0.2.0)](https://docs.github.com/en/github/managing-security- vulnerabilities/about-dependabot-security-updates#about-compatibility- scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> URL: sagemath#37945 Reported by: dependabot[bot] Reviewer(s):

Fixes sagemath#37998 (comment) - Split out from sagemath#37998 ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38034 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

Moving the letterplace implementation of free algebras to the `Parent` framework ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38053 Reported by: Frédéric Chapoton Reviewer(s): Frédéric Chapoton, Matthias Köppe, Travis Scrimshaw

### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38077 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

… files of existing distributions sagemath-{categories,...,tdlib}       These directives at the top of the file inform developers about the assignment of modules to pip-installable distributions. As of this PR, there should be no change to the existing distributions (sagemath-categories...) nor the monolithic build of the Sage library. This is a portion of: - sagemath#37901 In contrast to that PR, it has no dependencies. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38088 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

Moved the file set_labels_by_changes.sh to to the .ci directory. The workflow runs from the main repo. Fixed bug, the correct labels will be added even if the feature branch is behind. sagemath#37262 sagemath#37262 (comment) https://groups.google.com/g/sage-devel/c/w4IeYgXgVUc/m/Og7NP-3VAQAJ  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. URL: sagemath#38114 Reported by: Aman Moon Reviewer(s): Matthias Köppe

which I think are already established. But it is good to recommend preferred styles to keep docstrings in consistent styles. motivated by the comment: sagemath#37821 (comment) ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [x] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    sagemath#38068 (merged here) URL: sagemath#38117 Reported by: Kwankyu Lee Reviewer(s): gmou3, Kwankyu Lee

Weyl algebras elements cannot be hashed or compared, since the element class is too barren. This patch lets the element class inherit from IndexedFreeModuleElement, which fixes this problem automatically. URL: sagemath#38119 Reported by: Darij Grinberg Reviewer(s): Travis Scrimshaw

@jhpalmieri

…ats for stats       Hopefully fixes sagemath#38116 @jhpalmieri ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38122 Reported by: Matthias Köppe Reviewer(s):

…Field`, `is_RelativeNumberField`, `is_NumberFieldIdeal`, `is_NumberFieldFractionalIdeal`, `is_NumberFieldFractionalIdeal_rel`       ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38124 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee, Matthias Köppe

### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38125 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

…lection`, `is_ToricLattice`, `is_ToricLatticeElement`, `is_ToricLatticeQuotient`       ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38126 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

solves some items of wishlist in sagemath#37759 ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38127 Reported by: Kwankyu Lee Reviewer(s): Matthias Köppe

simplify code in various places by using more list comprehension, as suggested by `ruff --select=PERF4` ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38131 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

…numerical/ fixing pycodestyle warning for blank lines in pyx files in `groups` and `numerical` folders ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38132 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

fixing pycodestyle warning about blank lines in pyx files in `coding` and `modules` ### 📝 Checklist - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. URL: sagemath#38133 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

fixing pycodestyle warning for blank lines in pyx files in matrix folder ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38134 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

…SE, AVX Fixes sagemath#38000 Upstream PR at linbox-team/fflas-ffpack#397 URL: sagemath#38136 Reported by: Volker Braun Reviewer(s):

some care for blank lines in pyx files in `rings` (only partial here) ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38137 Reported by: Frédéric Chapoton Reviewer(s): Matthias Köppe

- Cherry-picked from sagemath#35095. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38143 Reported by: Matthias Köppe Reviewer(s): David Coudert

CI Fix to fix the doctest dataflow warning that shows in all PRs: sagemath#38143 (review) ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38146 Reported by: Matthias Köppe Reviewer(s): David Coudert

As suggested by mkoeppe

Co-authored-by: Travis Scrimshaw <[email protected]>

Suggested by reviewer kwankyu This reverts commit a3cee27.

…modules from sage.modules       - Rebased version of sagemath#34455 - Part of sagemath#30309 Fixes sagemath#34455 ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies    URL: sagemath#37526 Reported by: Matthias Köppe Reviewer(s): Matthias Köppe, Travis Scrimshaw

…t_digraph` Fixes sagemath#37642. We add parameter `sort_edges` to method `init_short_digraph` of the `short_digraph` data structure defined in `src/sage/graphs/base/static_sparse_graph.pxd|pyx`. - When set to `True` (default), for each vertex, the list of neighbors is sorted by increasing vertex labels. This enables to search for a vertex in this list using binary search, and so in time `O(log(n))`, but the overall running time of method `init_short_digraph` is in `O(m + n*log(m))`. - When set to `False`, neighbors are not sorted. The running time of method `init_short_digraph` is reduced to `O(n + m)`, but the running time for searching in the list of neighbors increases to `O(m)`. So this new parameter is particularly useful for linear time algorithms such as `lex_BFS`. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [x] I have updated the documentation accordingly. ### ⌛ Dependencies    URL: sagemath#37662 Reported by: David Coudert Reviewer(s):

… dimensional Lie algebra       We implement the algorithms in de Graaf to compute the solvable and nilradicals of a finite dimensional Lie algebra. To do so, we also make further improvements to how subalgebras and ideals of Lie algebras are constructed and handled. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [x] I have created tests covering the changes. - [x] I have updated the documentation accordingly. ### ⌛ Dependencies    - sagemath#37693 Uses this in the computation - sagemath#37682 Needs the improvements here URL: sagemath#37697 Reported by: Travis Scrimshaw Reviewer(s): Matthias Köppe, Travis Scrimshaw

Reverted changes made in sagemath#36986 The bug in sagemath#36974 remains unsolved. Efforts are being made in sagemath#37744 to resolve the bug. Fixes sagemath#37744 ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [x] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies URL: sagemath#37850 Reported by: Aman Moon Reviewer(s):

…modules       We provide an implementation of Garsia-Procesi modules, which are an important class of representations of the symmetric group that comes from the cohomology of the Springer fiber. This provides an implementation as a representation on a quotient ring. This also does the following as part of getting the code to work (and a few related changes): - Implement the meataxe algorithm over finite fields to compute the composition series. - Move the (Brauer) character and other methods from symmetric group representation to the general representation class. - Made the abstract representation class work more generically and streamlined the implementation. - Expose the `Representation` class as part of the semigroup (algebras) API. ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [x] I have created tests covering the changes. - [x] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#37871 Reported by: Travis Scrimshaw Reviewer(s): Matthias Köppe, Travis Scrimshaw

….0 to 0.2.0 Bumps [dev-hanz-ops/install-gh-cli-action](https://github.com/dev-hanz- ops/install-gh-cli-action) from 0.1.0 to 0.2.0. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/dev-hanz-ops/install-gh- cli-action/releases">dev-hanz-ops/install-gh-cli-action's releases</a>.</em></p> <blockquote> <h2>v0.2.0 - update to node20</h2> <ul> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/8fff9050dae2d81b38f94500d8b74ad1d1d47410">https://github.com/dev-hanz- ops/install-gh-cli- action/commit/8fff9050dae2d81b38f94500d8b74ad1d1d47410</a> (update to node20)</li> </ul> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/8fff9050dae2d81b38f94500d8b74ad1d1d47410"><code>8fff905</code></a> update to node 20</li> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/67bfd0539397d168c77ba70eac2647fef7f1f020"><code>67bfd05</code></a> README: only linux amd64</li> <li><a href="https://github.com/dev-hanz-ops/install-gh-cli-action/commi t/44745680363a7ba976f907c6d1b697a9fc311cf7"><code>4474568</code></a> Update readme with working example (<a href="https://redirect.github.com/dev-hanz-ops/install-gh-cli- action/issues/3">sagemath#3</a>)</li> <li>See full diff in <a href="https://github.com/dev-hanz-ops/install- gh-cli-action/compare/v0.1.0...v0.2.0">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot- badges.githubapp.com/badges/compatibility_score?dependency-name=dev- hanz-ops/install-gh-cli-action&package-manager=github_actions&previous- version=0.1.0&new- version=0.2.0)](https://docs.github.com/en/github/managing-security- vulnerabilities/about-dependabot-security-updates#about-compatibility- scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> URL: sagemath#37945 Reported by: dependabot[bot] Reviewer(s):

Fixes sagemath#37998 (comment) - Split out from sagemath#37998 ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38034 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

Moving the letterplace implementation of free algebras to the `Parent` framework ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. URL: sagemath#38053 Reported by: Frédéric Chapoton Reviewer(s): Frédéric Chapoton, Matthias Köppe, Travis Scrimshaw

Especially for internal functions (starting with `_`). The PR includes many changes, most of which are algorithmically inconsequential (simple type declarations and docstring edits). Careful review is mostly needed for commit `f0fd595`, where some extra type castings are done within basic `matroid` functions. ### ⌛ Dependencies - Depends on sagemath#38056. URL: sagemath#38057 Reported by: gmou3 Reviewer(s): gmou3, Matthias Köppe, Travis Scrimshaw

### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38077 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

… files of existing distributions sagemath-{categories,...,tdlib}       These directives at the top of the file inform developers about the assignment of modules to pip-installable distributions. As of this PR, there should be no change to the existing distributions (sagemath-categories...) nor the monolithic build of the Sage library. This is a portion of: - sagemath#37901 In contrast to that PR, it has no dependencies. ### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38088 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee

### 📝 Checklist  - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    URL: sagemath#38103 Reported by: Matthias Köppe Reviewer(s): gmou3, grhkm21, Matthias Köppe

Moved the file set_labels_by_changes.sh to to the .ci directory. The workflow runs from the main repo. Fixed bug, the correct labels will be added even if the feature branch is behind. sagemath#37262 sagemath#37262 (comment) https://groups.google.com/g/sage-devel/c/w4IeYgXgVUc/m/Og7NP-3VAQAJ  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. URL: sagemath#38114 Reported by: Aman Moon Reviewer(s): Matthias Köppe

which I think are already established. But it is good to recommend preferred styles to keep docstrings in consistent styles. motivated by the comment: sagemath#37821 (comment) ### 📝 Checklist  - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [x] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies    sagemath#38068 (merged here) URL: sagemath#38117 Reported by: Kwankyu Lee Reviewer(s): gmou3, Kwankyu Lee

Weyl algebras elements cannot be hashed or compared, since the element class is too barren. This patch lets the element class inherit from IndexedFreeModuleElement, which fixes this problem automatically. URL: sagemath#38119 Reported by: Darij Grinberg Reviewer(s): Travis Scrimshaw

@jhpalmieri