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

Scalaz deriving benchmarks #45

Merged
merged 2 commits into from
Nov 22, 2017
Merged

Scalaz deriving benchmarks #45

merged 2 commits into from
Nov 22, 2017

Conversation

fommil
Copy link
Contributor

@fommil fommil commented Nov 22, 2017

magnolia/show_50:
00:04.817789   src/adt.scala
00:08.817326   src/magnolia/show_50.scala
00:04.461259   src/magnolia/show_50.scala
00:03.224818   src/magnolia/show_50.scala
00:03.061996   src/magnolia/show_50.scala
00:02.791182   src/magnolia/show_50.scala
00:02.858702   src/magnolia/show_50.scala
00:02.576011   src/magnolia/show_50.scala
00:02.365986   src/magnolia/show_50.scala
00:02.682295   src/magnolia/show_50.scala
00:01.966328   src/magnolia/show_50.scala

kittens/show_50:
00:04.892484   src/adt.scala
01:03.009512   src/kittens/show_50.scala
00:50.582824   src/kittens/show_50.scala
00:49.491663   src/kittens/show_50.scala
00:53.874661   src/kittens/show_50.scala
00:56.904880   src/kittens/show_50.scala
00:57.479772   src/kittens/show_50.scala
00:51.768821   src/kittens/show_50.scala
00:54.901133   src/kittens/show_50.scala
00:56.331246   src/kittens/show_50.scala
01:00.491937   src/kittens/show_50.scala

scalaz-deriving/show_50:
00:06.517868   src/adt1.scala
00:00.562657   src/scalaz-deriving/show_50.scala
00:00.324758   src/scalaz-deriving/show_50.scala
00:00.252164   src/scalaz-deriving/show_50.scala
00:00.235869   src/scalaz-deriving/show_50.scala
00:00.230426   src/scalaz-deriving/show_50.scala
00:00.201479   src/scalaz-deriving/show_50.scala
00:00.210404   src/scalaz-deriving/show_50.scala
00:00.190581   src/scalaz-deriving/show_50.scala
00:00.173147   src/scalaz-deriving/show_50.scala
00:00.180840   src/scalaz-deriving/show_50.scala

@fommil
Copy link
Contributor Author

fommil commented Nov 22, 2017

I realised scalaz-deriving is also deriving Equal in this benchmark. Just doing Show shaves off a second

scalaz-deriving/show_50:
00:05.477280   src/adt1.scala
00:00.557310   src/scalaz-deriving/show_50.scala
00:00.314377   src/scalaz-deriving/show_50.scala
00:00.212032   src/scalaz-deriving/show_50.scala
00:00.189217   src/scalaz-deriving/show_50.scala
00:00.189498   src/scalaz-deriving/show_50.scala
00:00.165964   src/scalaz-deriving/show_50.scala
00:00.176364   src/scalaz-deriving/show_50.scala
00:00.181836   src/scalaz-deriving/show_50.scala
00:00.194030   src/scalaz-deriving/show_50.scala
00:00.179104   src/scalaz-deriving/show_50.scala

Copy link
Collaborator

@propensive propensive left a comment

Choose a reason for hiding this comment

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

Looks like it's moving in the right direction!

@propensive propensive merged commit 192173d into softwaremill:master Nov 22, 2017
@fommil fommil deleted the scalaz-deriving branch November 22, 2017 23:05
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

Successfully merging this pull request may close these issues.

None yet

2 participants