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

linsolve fails #357

Closed
melonedo opened this issue Jun 4, 2020 · 1 comment · Fixed by #358
Closed

linsolve fails #357

melonedo opened this issue Jun 4, 2020 · 1 comment · Fixed by #358

Comments

@melonedo
Copy link

melonedo commented Jun 4, 2020

When I was trying the tutorials in examples/solvers.md, I find these are probably broken.
solve seems to be good:

julia> @vars x y z
julia> solve([x + y + z - 1, x + y + 2*z - 3 ], (x, y, z))
Dict{Any,Any} with 2 entries:
  z => 2
  x => -y - 1

linsolve produces an empty set (this is also shown in the notebook version of tutorial):

julia> linsolve([x + y + z - 1, x + y + 2*z - 3 ], (x, y, z))
∅

solveset can't even run:

solveset([x + y + z - 1, x + y + 2*z - 3 ], (x, y, z))
ERROR: MethodError: no method matching solveset(::Array{Sym,1}, ::Tuple{Sym,Sym,Sym})
Closest candidates are:
  solveset(::SymPy.SymbolicObject, ::Any...; kwargs...) at C:\Users\melonedo\.julia\packages\SymPy\QNFlL\src\importexport.jl:96
Stacktrace:
 [1] top-level scope at REPL[17]:1

Are there any caveats in using this module?

@jverzani
Copy link
Collaborator

jverzani commented Jun 4, 2020

Thanks for pointing these out. I need to integrate the examples into the testing framework, as changes to the underlying sympy package can necessitate changes to this package. I'll look into these specific issues and address. There certainly have been changes.

@jverzani jverzani mentioned this issue Jun 5, 2020
2 tasks
jverzani added a commit that referenced this issue Jun 9, 2020
* setup up documenter

* add documenter documentation

* remove examples, as this was moved  to   docs

* edit docs, docstrings; tweak subs;tweak .travisyml
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 a pull request may close this issue.

2 participants