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

Extract recording #1245

Merged
merged 56 commits into from
Nov 23, 2023
Merged

Extract recording #1245

merged 56 commits into from
Nov 23, 2023

Conversation

fikovnik
Copy link
Collaborator

No description provided.

@fikovnik fikovnik changed the title Extract recording WIP - Extract recording Jul 11, 2023
@fikovnik fikovnik force-pushed the extract-recording branch from 140d320 to a24e34a Compare July 13, 2023 12:26
@fikovnik fikovnik force-pushed the extract-recording branch from f6e9228 to 799bb95 Compare July 20, 2023 13:29
@o-
Copy link
Contributor

o- commented Jul 20, 2023

Here are some stats for your PR:

  • WARNING: the longest CI job test_release_2 took 2.42h
  • shootout suite regressed by 0.95
  • simple suite regressed by 0.95
  • summary suite regressed by 0.96
  • Bounce_nonames_simple regressed by 0.92
  • spectralnorm_alt_3 regressed by 0.34
  • pidigits regressed by 0.63
  • fastaredux regressed by 0.89
  • fastaredux_naive regressed by 0.95
  • profiler-shared regressed by 0.61
  • scalarFor regressed by 0.84
  • emptyFor regressed by 0.92
  • profiler-microbenchmark regressed by 0.93
  • Overall benchmarks regressed by 0.96

Please find your performance results at https://rir-benchmarks.prl.fit.cvut.cz/diff?job_ids[]=4701404478&job_ids[]=4662886663&selection=all

Copy link
Collaborator

@skrynski skrynski left a comment

Choose a reason for hiding this comment

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

there are a few FIXME still, FeedbackOrigin to FeedbackPosition for example

@o-
Copy link
Contributor

o- commented Aug 28, 2023

Here are some stats for your PR:

  • WARNING: the longest CI job test_release_2 took 2.51h
  • pidigits regressed by 0.62
  • reversecomplement regressed by 0.91
  • mandelbrot_naive_ascii regressed by 0.93
  • scalarFor regressed by 0.83
  • Overall benchmarks regressed by 0.99

Please find your performance results at https://rir-benchmarks.prl.fit.cvut.cz/diff?job_ids[]=4966931473&job_ids[]=4960940727&selection=all

@o-
Copy link
Contributor

o- commented Aug 30, 2023

Here are some stats for your PR:

  • WARNING: the longest CI job test_release_2 took 3.13h
  • are-we-fast-r suite improved by 1.03
  • pidigits regressed by 0.62
  • fasta_2 regressed by 0.92
  • binarytrees_2 regressed by 0.95
  • scalarFor regressed by 0.82
  • emptyFor regressed by 0.86
  • Overall benchmarks regressed by 0.99

Please find your performance results at https://rir-benchmarks.prl.fit.cvut.cz/diff?job_ids[]=4975615535&job_ids[]=4960940727&selection=all

@o-
Copy link
Contributor

o- commented Aug 31, 2023

Here are some stats for your PR:

  • WARNING: the longest CI job test_release_2 took 2.4h
  • flexclust regressed by 0.94
  • spectralnorm_naive regressed by 0.88
  • fasta_2 regressed by 0.93
  • nbody_2 regressed by 0.95
  • nbody_3 regressed by 0.95
  • emptyFor regressed by 0.83
  • Overall benchmarks regressed by 1.0

Please find your performance results at https://rir-benchmarks.prl.fit.cvut.cz/diff?job_ids[]=4989052594&job_ids[]=4960940727&selection=all

@o-
Copy link
Contributor

o- commented Nov 22, 2023

Here are some stats for your PR:

  • WARNING: the longest CI job test_release_2 took 2.49h
  • are-we-fast-r suite improved by 1.03
  • spectralnorm_alt_3 regressed by 0.81
  • reversecomplement regressed by 0.93
  • profiler-microbenchmark regressed by 0.89
  • emptyFor regressed by 0.94
  • Overall benchmarks regressed by 1.0

Please find your performance results at https://rir-benchmarks.prl.fit.cvut.cz/diff?job_ids[]=5594390056&job_ids[]=5585852464&selection=all

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.

3 participants