Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

expand and explain doctest of _preprocess_intertwining_relations #39310

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

mantepse
Copy link
Collaborator

@mantepse mantepse commented Jan 10, 2025

This is related to #39307. I need to understand precisely how manipulation of _P, _compute_possible_block_values, etc. interact, and this is a tiny step in this direction.

Copy link

github-actions bot commented Jan 10, 2025

Documentation preview for this PR (built with commit 7d53e28; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

Copy link
Contributor

@dcoudert dcoudert left a comment

Choose a reason for hiding this comment

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

LGTM.

vbraun pushed a commit to vbraun/sage that referenced this pull request Jan 19, 2025
…ing_relations

    
This is related to sagemath#39307.  I need to understand precisely how
manipulation of `_P`, `_compute_possible_block_values`, etc. interact,
and this is a tiny step in this direction.
    
URL: sagemath#39310
Reported by: Martin Rubey
Reviewer(s): David Coudert, Martin Rubey
vbraun pushed a commit to vbraun/sage that referenced this pull request Jan 20, 2025
…ing_relations

    
This is related to sagemath#39307.  I need to understand precisely how
manipulation of `_P`, `_compute_possible_block_values`, etc. interact,
and this is a tiny step in this direction.
    
URL: sagemath#39310
Reported by: Martin Rubey
Reviewer(s): David Coudert, Martin Rubey
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.

2 participants