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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub actions #322

Merged
merged 3 commits into from Jul 6, 2020
Merged

GitHub actions #322

merged 3 commits into from Jul 6, 2020

Conversation

PragTob
Copy link
Member

@PragTob PragTob commented Jul 5, 2020

This is #321 but pushed by me, so maybe/hopefully it works? 馃

@pablocostass

@PragTob
Copy link
Member Author

PragTob commented Jul 5, 2020

Well it's running that's something :D

Copy link
Member Author

@PragTob PragTob left a comment

Choose a reason for hiding this comment

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

Looks good @pablocostass

Let's see it all pass then I'll deactivate appveyor and exchange the build badges :)

uses: actions/checkout@v2
- name: Install Elixir
run: |
cinst elixir
Copy link
Member Author

Choose a reason for hiding this comment

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

interesting that this would work here as it's exactly what I did over at appveyor 馃

Copy link
Contributor

Choose a reason for hiding this comment

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

Magic is real I guess 馃し

@@ -0,0 +1 @@
# Dialyzer needs the directory to exist beforehand
Copy link
Member Author

Choose a reason for hiding this comment

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

馃憤

@PragTob
Copy link
Member Author

PragTob commented Jul 5, 2020

Grml probably overexertion:


  1) test .run_scenarios can run multiple benchmarks in parallel (Benchee.Benchmark.RunnerTest)
     test/benchee/benchmark/runner_test.exs:82
     Assertion with >= failed
     code:  assert length(run_times_for(new_suite, "")) >= 8
     left:  3
     right: 8
     stacktrace:
##[error]       test/benchee/benchmark/runner_test.exs:93: (test)

I'm too tired.. probably gotta retry that one more or make it more lenient :|

@pablocostass
Copy link
Contributor

@PragTob Hang in there 馃挭

If after a re-run it still fails I can, as you suggested, make that test more forgiving :)

@PragTob
Copy link
Member Author

PragTob commented Jul 6, 2020

@pablocostass I'll take a look at it right now :)

@PragTob
Copy link
Member Author

PragTob commented Jul 6, 2020

wow, after all that caching it is quite a bit faster!

But now it started failing on mac for... different reasons 馃槩

   1) test .run_scenarios each triggers for every invocation, scenario once (Benchee.Benchmark.RunnerTest)
     test/benchee/benchmark/runner_test.exs:632
     Assertion with >= failed
     code:  assert hook_call_count >= 2
     left:  1
     right: 2
     stacktrace:
       test/benchee/benchmark/runner_test.exs:701: (test)

@PragTob
Copy link
Member Author

PragTob commented Jul 6, 2020


  1) test formatters have full access to the suite data, values in assigns (BencheeTest)
     test/benchee_test.exs:315
     Assertion with =~ failed
     code:  assert output =~ ~r"Run time: #{@rough_10_milli_s}$"m
     left:  "Operating System: macOS\nCPU Information: Intel(R) Xeon(R) CPU E5-1650 v2 @ 3.50GHz\nNumber of Available Cores: 4\nAvailable memory: 12 GB\nElixir 1.10.3\nErlang 23.0.2\n\nBenchmark suite executing with the following configuration:\nwarmup: 5 ms\ntime: 10 ms\nmemory time: 0 ns\nreduction time: 0 ns\nparallel: 1\ninputs: none specified\nEstimated total run time: 15 ms\n\nBenchmarking Sleeps...\nRun time: 20.47 ms\nAverage: 20.47 ms\nCustom value\n"
     right: ~r/Run time: ((8|9|10|11|12|13|14)\.\d{2} ms)$/m
     stacktrace:
       test/benchee_test.exs:351: anonymous fn/0 in BencheeTest."test formatters have full access to the suite data, values in assigns"/1
       test/benchee_test.exs:316: (test)

This one already has a retrying attached to it.

well... changed the timings

@PragTob
Copy link
Member Author

PragTob commented Jul 6, 2020

So.. that didn't work.

  1) test formatters have full access to the suite data, values in assigns (BencheeTest)
     test/benchee_test.exs:315
     Assertion with =~ failed
     code:  assert output =~ ~r"Run time: #{@rough_10_milli_s}$"m
     left:  "Operating System: macOS\nCPU Information: Intel(R) Xeon(R) CPU E5-1650 v2 @ 3.50GHz\nNumber of Available Cores: 4\nAvailable memory: 12 GB\nElixir 1.10.3\nErlang 23.0.2\n\nBenchmark suite executing with the following configuration:\nwarmup: 20 ms\ntime: 100 ms\nmemory time: 0 ns\nreduction time: 0 ns\nparallel: 1\ninputs: none specified\nEstimated total run time: 120 ms\n\nBenchmarking Sleeps...\nRun time: 20.10 ms\nAverage: 21.45 ms\nCustom value\n"
     right: ~r/Run time: ((8|9|10|11|12|13|14)\.\d{2} ms)$/m
     stacktrace:
       test/benchee_test.exs:351: anonymous fn/0 in BencheeTest."test formatters have full access to the suite data, values in assigns"/1
       test/benchee_test.exs:316: (test)

I'm... I can't. I bet it's the damn bad time measurement on Mac. I guess I'll have to deactivate Mac for these measurement sensitive ones. Or wait... one more shot.

@PragTob
Copy link
Member Author

PragTob commented Jul 6, 2020

Aight. this is it...

  1) test formatters have full access to the suite data, values in assigns (BencheeTest)
     test/benchee_test.exs:315
     Assertion with =~ failed
     code:  assert output =~ ~r"Average: #{@rough_30_milli_s}$"m
     left:  "Operating System: macOS\nCPU Information: Intel(R) Xeon(R) CPU E5-1650 v2 @ 3.50GHz\nNumber of Available Cores: 4\nAvailable memory: 12 GB\nElixir 1.10.3\nErlang 23.0.2\n\nBenchmark suite executing with the following configuration:\nwarmup: 60 ms\ntime: 120 ms\nmemory time: 0 ns\nreduction time: 0 ns\nparallel: 1\ninputs: none specified\nEstimated total run time: 180 ms\n\nBenchmarking Sleeps...\nRun time: 33.35 ms\nAverage: 41.30 ms\nCustom value\n"
     right: ~r/Average: ((28|29|30|31|32|33|34)\.\d{2} ms)$/m
     stacktrace:
       test/benchee_test.exs:352: anonymous fn/0 in BencheeTest."test formatters have full access to the suite data, values in assigns"/1
       test/benchee_test.exs:316: (test)

PragTob added a commit that referenced this pull request Jul 6, 2020
Too much trouble trying to get github macos tests to pass #322
@PragTob
Copy link
Member Author

PragTob commented Jul 6, 2020

Seems it needs to install elixir every time on windows and that takes some time... would be great if we could also cache that but not super high prio as it's kinda like before.

pablocostass and others added 3 commits July 6, 2020 19:14
This commit removes from the project both the Travis and AppVeyor CIs
and adds a directory needed to cache the PLTs from Dialyzer.
Too much trouble trying to get github macos tests to pass #322

But also before that, make specs more lenient overall so that
it's not random.
@PragTob PragTob merged commit d6348a9 into main Jul 6, 2020
@PragTob PragTob deleted the github-actions branch July 6, 2020 17:32
@pablocostass
Copy link
Contributor

Seems it needs to install elixir every time on windows and that takes some time... would be great if we could also cache that but not super high prio as it's kinda like before.

@PragTob I mistook cache and setup-elixir requirements and I thought the former couldn't be used in Windows when that's the case of the latter, oopsie. I'll open a PR with the change later.

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