Skip to content
This repository has been archived by the owner on Sep 9, 2024. It is now read-only.

example of a user defined type #107

Merged
merged 2 commits into from
Sep 1, 2016
Merged

Conversation

jkozdon
Copy link
Contributor

@jkozdon jkozdon commented Jul 11, 2016

No description provided.

@jkozdon
Copy link
Contributor Author

jkozdon commented Jul 11, 2016

There doesn't seem to be an example showing what's required for the user to define their own type.

@coveralls
Copy link

coveralls commented Jul 11, 2016

Coverage Status

Coverage remained the same at 93.651% when pulling e5c46a2 on jkozdon:master into a0ef97d on JuliaLang:master.

@codecov-io
Copy link

codecov-io commented Jul 11, 2016

Current coverage is 93.65%

No coverage report found for master at a0ef97d.

Powered by Codecov. Last updated by a0ef97d...71a1628

end

# Necessary for the solver
Base.(:/)(x::Vec2, y::Real) = Vec2(x.a/y, x.b/y)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To make this 0.5-proof: @compat Base.:+

@mauro3
Copy link
Contributor

mauro3 commented Jul 11, 2016

Thanks, this is good! Maybe a few sentences comments would be good: what the example is about, maybe a word about the ODE.

There is also the test-case test/interface-tests.jl. You could mention it in your example.

@coveralls
Copy link

coveralls commented Jul 14, 2016

Coverage Status

Coverage remained the same at 93.651% when pulling 63ad4b0 on jkozdon:master into a0ef97d on JuliaLang:master.

@mauro3 mauro3 mentioned this pull request Jul 18, 2016
@mauro3
Copy link
Contributor

mauro3 commented Jul 18, 2016

Looks good to me. Someone please merge this, unless there are objections.

@ChrisRackauckas
Copy link
Member

Looks good to me as well. Merging.

@ChrisRackauckas ChrisRackauckas merged commit f6fc008 into SciML:master Sep 1, 2016
@mauro3
Copy link
Contributor

mauro3 commented Sep 2, 2016

Ah, yes, we can merge now! ;-) Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants