-
-
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
[GSoC] Planar Joint #24046
[GSoC] Planar Joint #24046
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. |
@brocksam, @moorepants, @sidhu1012 FYI |
524882c
to
36410f9
Compare
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] [f002fe53]
<sympy-1.11.1^0>
- 987±2μs 632±1μs 0.64 solve.TimeSparseSystem.time_linear_eq_to_matrix(10)
- 2.82±0.01ms 1.18±0ms 0.42 solve.TimeSparseSystem.time_linear_eq_to_matrix(20)
- 5.67±0.02ms 1.72±0ms 0.30 solve.TimeSparseSystem.time_linear_eq_to_matrix(30)
Full benchmark results can be found as artifacts in GitHub Actions |
e177b59
to
9ef356e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like all combinations of possible user inputs are accounted for in the tests.
e5cd48f
to
70e43af
Compare
@brocksam Just updated the |
There was some recent editing to the |
Can you update with master? |
Did a rebase to the master and squased my commits, so the commit tree is not as messy with all those small commits. Sorry for the mess of deleting and restoring the branch. |
9ec29a7
to
207dbd8
Compare
After some discussing with @brocksam and @moorepants we came to the conclusion that it is best to leave the |
63e63df
to
3fb764e
Compare
Sounds good to me. Let me know when this one is ready and I can merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo/punctuation suggestions. Once corrected this is ready to be merged.
@moorepants, assuming that the tests still pass, this PR is ready for merging. |
References to other Issues or PRs
Part of #21519
Brief description of what is fixed or changed
Implements the PlanarJoint
Other comments
Release Notes