Skip to content

Conversation

jhpalmieri
Copy link
Member

@jhpalmieri jhpalmieri commented Oct 4, 2023

Deprecate the backslash operator.

Deprecate the backslash operator: it is underused and its current implementation conflicts with standard Python usage of \. A poll on sage-devel supports this decision.

The backslash operator is only used in a few places, mainly matrices and matroids; those uses are deprecated.

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • 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 accordingly.

⌛ Dependencies

@jhpalmieri jhpalmieri force-pushed the deprecate-backslash-operator branch 2 times, most recently from e5f2b90 to d2ad093 Compare October 4, 2023 04:54
@jhpalmieri jhpalmieri marked this pull request as ready for review October 4, 2023 04:58
@jhpalmieri jhpalmieri force-pushed the deprecate-backslash-operator branch from d2ad093 to 686cdd9 Compare October 4, 2023 20:32
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Documentation preview for this PR (built with commit 686cdd9; changes) is ready! 🎉

@kwankyu
Copy link
Collaborator

kwankyu commented Oct 8, 2023

Otherwise, lgtm.

Copy link
Collaborator

@kwankyu kwankyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

@jhpalmieri
Copy link
Member Author

Thank you!

@vbraun vbraun merged commit 41cf276 into sagemath:develop Oct 14, 2023
vbraun pushed a commit to vbraun/sage that referenced this pull request Aug 25, 2025
sagemathgh-40656: remove deprecated method in matroid.pyx
    
This was deprecated in sagemath#36394

### 📝 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.
    
URL: sagemath#40656
Reported by: Frédéric Chapoton
Reviewer(s): David Coudert
vbraun pushed a commit to vbraun/sage that referenced this pull request Sep 4, 2025
sagemathgh-40711: remove some deprecations in matrix2
    
after

- File: src/sage/matrix/matrix2.pyx ; PR: sagemath#36394 ; Closed Date:
2023-10-14
- File: src/sage/matrix/matrix2.pyx ; PR: sagemath#29243 ; Closed Date:
2020-08-30
- File: src/sage/matrix/matrix2.pyx ; PR: sagemath#31619 ; Closed Date:
2021-06-06

### 📝 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.
    
URL: sagemath#40711
Reported by: Frédéric Chapoton
Reviewer(s): David Coudert
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants