-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Solve diophantine documentation guide #24051
Conversation
✅ Hi, I am the SymPy bot (v167). I'm here to help you write a release notes entry. Please read the guide on how to write release notes. Your release notes are in good order. Here is what the release notes will look like:
This will be added to https://github.com/sympy/sympy/wiki/Release-Notes-for-1.12. Click here to see the pull request description that was parsed.
Update The release notes on the wiki have been updated. |
🟠Hi, I am the SymPy bot (v167). I've noticed that some of your commits add or delete files. Since this is sometimes done unintentionally, I wanted to alert you about it. This is an experimental feature of SymPy Bot. If you have any feedback on it, please comment at sympy/sympy-bot#75. The following commits add new files:
If these files were added/deleted on purpose, you can ignore this message. |
Benchmark results from GitHub Actions Lower numbers are good, higher numbers are bad. A ratio less than 1 Significantly changed benchmark results (PR vs master) Significantly changed benchmark results (master vs previous release) before after ratio
[41d90958] [c79d74da]
<sympy-1.11.1^0>
- 995±4μs 623±2μs 0.63 solve.TimeSparseSystem.time_linear_eq_to_matrix(10)
- 2.84±0ms 1.16±0ms 0.41 solve.TimeSparseSystem.time_linear_eq_to_matrix(20)
- 5.67±0.02ms 1.70±0ms 0.30 solve.TimeSparseSystem.time_linear_eq_to_matrix(30)
Full benchmark results can be found as artifacts in GitHub Actions |
Demonstrate creating a dictionary of symbol-expression pairs
For some reason unrelated to this pull request I believe, optional-dependencies (pypy-3.8, false) and optional-dependencies (3.11.0-alpha - 3.11, true) give errors on the same file: |
…to solve-diophantine
Use pythag instead of repeating a**2 + b**2 - c**2 throughout
I wonder if we should try to make the example blocks more self-contained here. Each block here cannot be copied and run on its own because it depends on variables defined in earlier blocks (like |
I just went through and did that. At the start of this project, I mistakenly thought all code blocks had to be self-contained, so I initially made them all self-contained. When I learned that wasn't the case, I relaxed that somewhat, and initially probably did too much of that on this page. Part of the issue is that you can't include comment-only lines in code blocks (doctest fails those tests), so I thought it made more sense to break related code into separate blocks so I could include more exposition before sections of code. |
Are you sure about that? I see lots of examples from |
It seems like comment-only lines in code blocks are allowed now, for example
Either I got confused about what caused a failure, or maybe the doctest code (criteria) changed... |
References to other Issues or PRs
Brief description of what is fixed or changed
Other comments
Release Notes