Skip to content

Benchmarks cleanup#4

Open
mikkolaj wants to merge 1 commit intoseries/3.x-avsfrom
cleanup-benchmarks
Open

Benchmarks cleanup#4
mikkolaj wants to merge 1 commit intoseries/3.x-avsfrom
cleanup-benchmarks

Conversation

@mikkolaj
Copy link
Copy Markdown

@mikkolaj mikkolaj commented Aug 5, 2025

Make running benchmarks easier:

  • remove outdated benchmarks
  • uncomment and fix compilation of benchmarks
  • comment out zio / cats benchmarks as they're not important when comparing monix versions

* uncomment and fix compilation of benchmarks
* comment out zio / cats benchmarks as they're not important when comparing monix versions
@halotukozak
Copy link
Copy Markdown
Member

just remove commented benchmarks

class ObservableMapTaskBenchmark {
@Param(Array("10000"))
var size: Int = _
var size: Long = _
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

?

@@ -1,158 +0,0 @@
/*
* Copyright (c) 2014-2021 by The Monix Project Developers.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why?

@@ -1,71 +0,0 @@
/*
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why?

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.

2 participants