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

Enumerate tutorials and rebuild #61

Closed
wants to merge 1 commit into from
Closed

Conversation

goggle
Copy link
Contributor

@goggle goggle commented Jun 27, 2019

I have renamed and rebuilt all the tutorials as suggested in #60.

Here are some problems, which I could not solve by installing the needed packages (I have not modified any of the *.jmd files in tutorials/ except of renaming them).

introduction/05-formatting-plots:
The line

plot(sol,vars=(:x,:y,:z))

leads to this error:

Error: MethodError: no method matching pointer(::Symbol)
Closest candidates are:
pointer(!Matched::String) at strings/string.jl:81
pointer(!Matched::String, !Matched::Integer) at strings/string.jl:82
pointer(!Matched::SubString{String}) at strings/substring.jl:104

models/06-pendulum_bayesian_inference gives some warnings when building with Weave.jl:

[ Info: Weaving chunk 8 from line 91
┌ Warning: seriestype density has been moved to StatsPlots.  To use: `Pkg.add("StatsPlots"); using StatsPlots`
└ @ Plots ~/.julia/packages/Plots/oiirH/src/args.jl:1061
┌ Warning: seriestype density has been moved to StatsPlots.  To use: `Pkg.add("StatsPlots"); using StatsPlots`
└ @ Plots ~/.julia/packages/Plots/oiirH/src/args.jl:1061
┌ Warning: seriestype density has been moved to StatsPlots.  To use: `Pkg.add("StatsPlots"); using StatsPlots`

I don't know if this is relevant...

ode_extras/01-ModelingToolkit: I'm unable to install PuMaS.jl:

using PuMaS
Error: ArgumentError: Package PuMaS not found in current path:
- Run `import Pkg; Pkg.add("PuMaS")` to install the PuMaS package.

import Pkg
Pkg.add("PuMaS")

ERROR: The following package names could not be resolved:
 * PuMaS (not found in project, manifest or registry)
Please specify by known `name=uuid`.

type_handling/01-number_types: There are two errors.

using DecFP
prob_ode_decfplinear = ODEProblem(f,Dec128(1)/Dec128(2),(Dec128(0.0),Dec128(1.0)),Dec128(1.01))
sol =solve(prob_ode_decfplinear,Tsit5())

gives

ERROR: StackOverflowError:

and

using Decimals
prob_ode_decimallinear = ODEProblem(f,[decimal("1.0")]./[decimal("2.0")],(0//1,1//1),decimal(1.01))
sol =solve(prob_ode_decimallinear,RK4(),dt=1/2^(6)) #Fails

gives

ERROR: MethodError: Decimals.Decimal(::Rational{Int64}) is ambiguous. Candidates:
  (::Type{T})(x::Rational{S}) where {S, T<:AbstractFloat} in Base at rational.jl:92
  Decimals.Decimal(num::Real) in Decimals at /home/alex/.julia/packages/Decimals/Qfcas/src/decimal.jl:13
Possible fix, define
  Decimals.Decimal(::Rational{S})

which seems to be somewhat expected from that #Fails comment.

type_handling/01-number_types gives two errors:
The line

t + sqrt(t)

results in

Error: DimensionError: 1.0 s and 1.0 s^1/2 are not dimensionally compatible.

And the code

using DifferentialEquations
f = (y,p,t) -> 0.5*y
u0 = 1.5u"N"
prob = ODEProblem(f,u0,(0.0u"s",1.0u"s"))
sol = solve(prob,Tsit5())

gives

Error: DimensionError: N s^-1 and 0.75 N are not dimensionally compatible.

Furthermore, I was unable to build the PDFs for the following tutorials because of LaTeX errors:

  • models/03-diffeqbio_I_introduction
  • models/04-diffeqbio_II_networkproperties
  • models/06-pendulum_bayesian_inference
  • models/07-outer_solar_system
    Is that something that should be reported to Weave.jl?

@goggle
Copy link
Contributor Author

goggle commented Jun 27, 2019

This PR looks messy... I think I will close it and open a new one.
I've just discovered that there is currently no Jupyter notebook for ode-extras/ModelingToolkit. Is there any reason for that?

@ChrisRackauckas
Copy link
Member

I've just discovered that there is currently no Jupyter notebook for ode-extras/ModelingToolkit. Is there any reason for that?

Because I was at a conference when I made it so I rushed it lol

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