-
Notifications
You must be signed in to change notification settings - Fork 454
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
Add Vector construct #97
Comments
The implementation for Array is simply not possible due to some limitations with non-reified generics. Let's consider porting a persistent vector from Scala, Clojure, or any external library instead. |
Some persistent Vector implementations: Kotlin based off PCollections: https://github.com/Kotlin/kotlinx.collections.immutable |
Co-authored-by: danieh <[email protected]>
@pakoito, thanks for bringing this up, looks like we decided in a different direction just supporting the std lib types. Closing this outdated issue. Thanks! |
https://github.com/FineCinnamon/Katz/blob/master/katz/src/main/kotlin/katz/data/NonEmptyList.kt
https://github.com/FineCinnamon/Katz/blob/master/katz/src/main/kotlin/katz/instances/NonEmptyListBimonad.kt
https://github.com/FineCinnamon/Katz/blob/master/katz/src/main/kotlin/katz/instances/NonEmptyListMonad.kt
https://github.com/FineCinnamon/Katz/blob/master/katz/src/main/kotlin/katz/instances/NonEmptyListComonad.kt
https://github.com/FineCinnamon/Katz/blob/master/katz/src/main/kotlin/katz/instances/NonEmptyListSemigroup.kt
It'll also require a Monoid instance, rather than a semigroup. You have to figure out why and how ;)
The text was updated successfully, but these errors were encountered: