Skip to content
This repository was archived by the owner on Nov 22, 2023. It is now read-only.

switch to StaticArrays and get 0.6 working [WIP] #80

Merged
merged 22 commits into from
May 4, 2017

Conversation

SimonDanisch
Copy link
Member

No description provided.

@SimonDanisch
Copy link
Member Author

This disables convexhulls and gjk for now, since they error and I don't really use that part of GeometryTypes.
I hope @jw3126 has time to look into those :)

@SimonDanisch
Copy link
Member Author

Also, seems like relying on StaticArrays makes GeometryTypes 0.6 only -.- ... not sure if I like that, but maybe reasonable, considering that people can tag a previous version for 0.5!

@jw3126
Copy link
Contributor

jw3126 commented Apr 6, 2017

I will fix the gjk stuff, when I switch to v0.6.

@jw3126
Copy link
Contributor

jw3126 commented Apr 18, 2017

Should I fork this branch or better wait until its merged with master?

@SimonDanisch
Copy link
Member Author

hm good question! I'm a bit hesitant with merging it, since it makes geometrytypes 0.6 only... But maybe it's time for that! Note that you'll need: JuliaArrays/StaticArrays.jl#138
I should just try to merge at least that into master as quick as possible!

@jw3126
Copy link
Contributor

jw3126 commented Apr 18, 2017

No rush from my side.

@sbromberger
Copy link

maybe it's time for that

Yes. It's time! :)

(I really miss being able to plot stuff in 0.6.)

@SimonDanisch SimonDanisch merged commit d33ccea into master May 4, 2017
@jw3126 jw3126 mentioned this pull request May 5, 2017
Merged
for (i,j) in enumerate(range)
result[i] = vec[mod1(j, length(vec))]
end
result[end] = endpoint
result
end


# Taken from Iterators.jl, since it's not possible to use Iterators.jl on 0.6 with precompilation
Copy link
Contributor

Choose a reason for hiding this comment

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

when you copy code, you need to respect the license of that code

Copy link
Member Author

Choose a reason for hiding this comment

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

oh, that was just supposed to be a local stop gap solution. Is there actually a fix by now? I just tried it right now and am still getting a precompile error on first precompile when importing Iterators....

Copy link
Contributor

Choose a reason for hiding this comment

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

try on rc3

Copy link
Member Author

Choose a reason for hiding this comment

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

ha can't reproduce it anymore :) I removed the code in #100

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