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

Additional benchmark functions #227

Merged
merged 1 commit into from Jun 25, 2015

Conversation

@robgarden
Copy link
Member

commented May 14, 2015

No description provided.

@robgarden robgarden force-pushed the robgarden:benchmarkAdditions branch from 4084689 to 5d78739 May 14, 2015

val genUrsemWaves =
(Gen.choose(-0.9, 1.2) |@| Gen.choose(-1.2, 1.2)) { Tuple2.apply }

This comment has been minimized.

Copy link
@gpampara

gpampara May 14, 2015

Member

All these genUrsm* should either be abstracted in some way to be used from other places or we should perhaps have them inline in the property definition?

@gpampara

This comment has been minimized.

Copy link
Member

commented May 14, 2015

We should consider splitting the file into smaller units, even if it means that the only reason we do it is to reduce compile times for the benchmarks and tests.

@gpampara

This comment has been minimized.

Copy link
Member

commented May 17, 2015

@robgarden any feedback on this yet?

@robgarden robgarden force-pushed the robgarden:benchmarkAdditions branch from 5d78739 to 04e44f5 Jun 24, 2015
gpampara added a commit that referenced this pull request Jun 25, 2015
Additional benchmark functions
@gpampara gpampara merged commit 77d63dc into cirg-up:series/2.0.x Jun 25, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.