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

Which is the right version to use? #18

Open
willowell opened this issue Oct 5, 2023 · 3 comments
Open

Which is the right version to use? #18

willowell opened this issue Oct 5, 2023 · 3 comments

Comments

@willowell
Copy link

Hello!

I'm interested in trying out SLASH, but I'm confused what the current version is - on Scaladex, I see 0.1 is the most recent version (https://index.scala-lang.org/dragonfly-ai/slash) but I see 0.101 on main (

val appVersion:String = "0.101"
) and on Scaladex under the old vector name (https://index.scala-lang.org/dragonfly-ai/slash/artifacts/vector?pre-releases=true).

Which one should I use?

Thank you!

@Quafadas
Copy link
Contributor

Quafadas commented Oct 8, 2023

@willowell

Hi, I'd recommend 0.1, it's the latest.

The docs should also point you at 0.1. I believe that line of code, to be outdated. Versioning is governed by the latest tag (you could track them if you wish), functionality provided courtesy of the excellent sbt-typelevel project.

Do post up in discussions how you get on. Feedback - good or bad, is welcome.

@Quafadas
Copy link
Contributor

Quafadas commented Oct 8, 2023

@dragonfly-ai

hey, for now, we disabled MIMA, and the appVersion line appears potentially confusing. If you agree, I'll put up a PR to move the tlBaseVersion to 0.1, and remove the appVersion attribute, which I think I would agree, is potentially confusing.

This is all governed by tags now and until if / when MIMA gets enabled, it's a no-op anyayz...

Quafadas added a commit to Quafadas/vector that referenced this issue Oct 8, 2023
@dragonfly-ai
Copy link
Owner

As of today, 0.2 is the best version to use.

It will require some refactoring, though, because every reference to ai.dragonfly.math has shortened to slash. Also if you used any mutable vector operations, they now return Unit instead of this.

Less significantly, the stream.{Beta, Binomial, Gaussian, LogNormal, Poisson} estimators no longer make any attempt toward concurrency safety. Please feel free to wrap these with a concurrency model of your choice.

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

No branches or pull requests

3 participants