declension / elm-obj-loader

An elm library for importing wavefront .obj files into your WebGL application.

Home Page:http://package.elm-lang.org/packages/declension/elm-obj-loader/latest

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Upgrade to Elm 0.19

declension opened this issue · comments

Hi. Thanks for this package, keen to get it working (with Elm 0.19)! Not sure what the particular reasons are, but:

$ elm install Zinggi/elm-obj-loader

produces:

The following packages do not work with Elm 0.19.0 right now:

    Zinggi/elm-obj-loader

Would be happy to help (where I can, not an Elm or WebGL expert...)

Just tried elm-upgrade, it did well, but:

WARNING! 2 of your dependencies have not yet been upgraded to
support Elm 0.19.
  - https://github.com/Bogdanp/elm-combine
  - https://github.com/Skinney/elm-array-exploration

Hey there.
I haven't had the time to upgrade this for elm 0.19 and I probably wont have it any time soon.

If your interested in getting this upgraded, I can give you some support:

  • elm-array-exploration: Will not be needed in 0.19, just replace it by the standard Array in the core library. They are the same now.
  • elm-combine: This was the parsing library I used. It hasn't been upgraded and it never will, as it used many custom operators which are no longer definable for package authors in 0.19. It would have to be replaced with elm/parser. Luckily I have a branch with an implementation using a very early version of elm/parser (https://github.com/Zinggi/elm-obj-loader/tree/progressBar)

Thanks for the pointers. I actually started down this path on Friday.

elm-array-exploration: Will not be needed in 0.19, just replace it by the standard Array in the core library. They are the same now.

Yep did that, no problem

elm-combine: This was the parsing library I used. It hasn't been upgraded and it never will, as it used many custom operators which are no longer definable for package authors in 0.19.

There's a fork of elm-combine which has 0.19 support, so I started using that. Obviously without all the custom operators it was a lot of work rewriting the elm-obj-loader parser but I think it's mostly there now. Of course, maybe the now-native parser was the better way to go, agh. I'll see.

I've left the elm/http on version 1.0, too much work to do that too :)

Are there any tests to help me prove it's working though? When I tried compiling the examples, they have further 0.19 compatibility issues so I'm not sure if those would be that useful (in time).

Thanks!

Thanks for putting so much work into this, it's very much appreciated.
Unfortunately there are no tests, I only have the examples 😞
Hope you still get it running.