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

update perturbation example #1292

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

karlwessel
Copy link
Contributor

This updates the perturbation example to use a more recent feature set of Symbolics.jl. Namely

  • Symbolics.coeff replaces most of the code in collect_powers and
  • symbolic_linear_solve is used instead of solve_for.

It also fixes some minor problems:

  • the link to the perturbation example on differential equations was dead
  • and the code on how to order the solution of the Kepler-equation by orders of $M$ was wrong.

If #1291 is fixed, one could also replace solve_coef completely with symbolic_solve using Groebner.

@karlwessel
Copy link
Contributor Author

The failing integration tests don't seem to be related to the PR.

@ChrisRackauckas
Copy link
Member

Oh sorry I thought I already merged this.

@ChrisRackauckas ChrisRackauckas merged commit 73cb6fa into JuliaSymbolics:master Oct 4, 2024
6 of 8 checks passed
@hersle hersle mentioned this pull request Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants