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.

Sign up for GitHub

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

Jump to bottom

Embedded sample CI bug #1070

Open
paulfd opened this issue Jan 15, 2022 · 3 comments
Open

Embedded sample CI bug #1070

paulfd opened this issue Jan 15, 2022 · 3 comments
Labels
bug Something isn't working ci continuous integration related
Milestone
1.3.0

Comments

@paulfd
Copy link
Member

paulfd commented Jan 15, 2022

The embedded sample test fails on Github windows CI. I can't reproduce it on my local machines.
@paulfd paulfd added this to the 1.3.0 milestone Jan 15, 2022
@paulfd paulfd added the bug Something isn't working label Jan 15, 2022
@paulfd
Copy link
Member Author

paulfd commented Jan 17, 2022

Actually bugged on linux once too.

@paulfd paulfd changed the title Windows embedded sample CI bug Embedded sample CI bug Jan 17, 2022
@redtide
Copy link
Member

redtide commented May 21, 2023 •
edited

It seems to build now (here), though there are some odd "warnings":
1. Failed to locate test files at the end of the Build Tests step
2. Parse error in "." at line 1: Cannot open file for reading: in Run Tests
3. CTests returns a No tests were found!!!, though it not fails.

IDK if these are about resources or built binaries.

redtide added a commit to redtide/sfizz that referenced this issue May 22, 2023
@redtide
CI update
e6e4684 
- Build the embedded sample test, fixes sfztools#1070
- Make bash|pwsh default shells and github.workspace as default working-directory
- Use ubuntu 22.04 for the Linux build to use abseil package when not building releases
- CTest working in the major OS builds
@redtide redtide mentioned this issue May 22, 2023
Merged
redtide added a commit to redtide/sfizz that referenced this issue May 22, 2023
@redtide
CI update
be45327 
- Build the embedded sample test, fixes sfztools#1070
- Make bash|pwsh default shells and github.workspace as default working-directory
- Use ubuntu 22.04 for the Linux build to use abseil package when not building releases
- CTest working in the major OS builds
@redtide
Copy link
Member

redtide commented May 22, 2023

I spoke too soon, it failed also on mac

Click here for log 
/Users/runner/work/sfizz/sfizz/tests/FilesT.cpp:811: FAILED:
  REQUIRE( tmp1 == tmp2 )
with expansion:
  { 0.00119f, 0.00121f, 0.00122f, 0.00122f, 0.00125f, 0.00126f, 0.00126f, 0.
  00128f, 0.00128f, 0.00129f, 0.00131f, 0.00131f, 0.00133f, 0.00135f, 0.00132f,
  0.00136f, 0.00134f, 0.00136f, 0.00139f, 0.00137f, 0.0014f, 0.0014f, 0.00141f,
  0.00143f, 0.00143f, 0.00144f, 0.00144f, 0.00145f, 0.00146f, 0.00146f, 0.
  00148f, 0.00149f, 0.00147f, 0.0015f, 0.00149f, 0.00151f, 0.00151f, 0.00151f,
  0.00152f, 0.00152f, 0.00153f, 0.00153f, 0.00153f, 0.00154f, 0.00154f, 0.
  00153f, 0.00156f, 0.00154f, 0.00156f, 0.00156f, 0.00156f, 0.00156f, 0.00157f,
  0.00155f, 0.00158f, 0.00158f, 0.00155f, 0.00158f, 0.00155f, 0.00158f, 0.
  00157f, 0.00159f, 0.00155f, 0.00158f, 0.00156f, 0.00155f, 0.00156f, 0.00155f,
  0.00155f, 0.00155f, 0.00154f, 0.00154f, 0.00153f, 0.00155f, 0.00151f, 0.
  00155f, 0.00151f, 0.00152f, 0.00153f, 0.00151f, 0.00152f, 0.00152f, 0.00148f,
  0.0015f, 0.00148f, 0.00149f, 0.00147f, 0.00147f, 0.00146f, 0.00144f, 0.
  00145f, 0.00144f, 0.00143f, 0.00143f, 0.0014f, 0.0014f, 0.00139f, 0.00138f,
  0.00138f, 0.00138f, 0.00137f, 0.00138f, 0.00136f, 0.00134f, 0.00134f, 0.
  00133f, 0.00132f, 0.00131f, 0.00128f, 0.00129f, 0.00126f, 0.00126f, 0.00127f,
  0.00124f, 0.00122f, 0.00121f, 0.0012f, 0.0012f, 0.00119f, 0.00118f, 0.00116f,
  0.00117f, 0.00111f, 0.00115f, 0.00109f, 0.0011f, 0.00109f, 0.00107f, 0.
  00106f, 0.00105f, 0.00103f, 0.00102f, 0.001f, 0.00099f, 0.00099f, 0.00096f,
  0.00095f, 0.00093f, 0.00092f, 0.00093f, 0.0009f, 0.00087f, 0.00087f, 0.
  00083f, 0.00083f, 0.00081f, 0.00079f, 0.00078f, 0.00078f, 0.00074f, 0.00073f,
  0.00071f, 0.00069f, 0.00069f, 0.00067f, 0.00064f, 0.00063f, 0.00061f, 0.
  00061f, 0.00058f, 0.00057f, 0.00053f, 0.00053f, 0.0005f, 0.00049f, 0.00048f,
  0.00046f, 0.00045f, 0.00044f, 0.00042f, 0.00041f, 0.00038f, 0.00038f, 0.
  00036f, 0.00033f, 0.00033f, 0.0003f, 0.00028f, 0.00027f, 0.00025f, 0.00025f,
  0.00022f, 0.00022f, 0.00019f, 0.00018f, 0.00015f, 0.00014f, 0.00013f, 0.
  00012f, 0.0001f, 0.00009f, 0.00007f, 0.00003f, 0.00005f, 0.00002f, 0.00001f,
  0.0f, -0.00001f, -0.00003f, -0.00005f, -0.00007f, -0.00009f, -0.0001f, -0.
  00012f, -0.00014f, -0.00015f, -0.00018f, -0.00019f, -0.0002f, -0.00022f, -0.
  00022f, -0.00025f, -0.00029f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f,
  0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f,
  0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f,
  0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f }
  ==
  { 0.00119f, 0.00121f, 0.00122f, 0.00122f, 0.00125f, 0.00126f, 0.00126f, 0.
  00128f, 0.00128f, 0.00129f, 0.00131f, 0.00131f, 0.00133f, 0.00135f, 0.00132f,
  0.00136f, 0.00134f, 0.00136f, 0.00139f, 0.00137f, 0.0014f, 0.0014f, 0.00141f,
  0.00143f, 0.00143f, 0.00144f, 0.00144f, 0.00145f, 0.00146f, 0.00146f, 0.
  00148f, 0.00149f, 0.00147f, 0.0015f, 0.00149f, 0.00151f, 0.00151f, 0.00151f,
  0.00152f, 0.00152f, 0.00153f, 0.00153f, 0.00153f, 0.00154f, 0.00154f, 0.
  00153f, 0.00156f, 0.00154f, 0.00156f, 0.00156f, 0.00156f, 0.00156f, 0.00157f,
  0.00155f, 0.00158f, 0.00158f, 0.00155f, 0.00158f, 0.00155f, 0.00158f, 0.
  00157f, 0.00159f, 0.00155f, 0.00158f, 0.00156f, 0.00155f, 0.00156f, 0.00155f,
  0.00155f, 0.00155f, 0.00154f, 0.00154f, 0.00153f, 0.00155f, 0.00151f, 0.
  00155f, 0.00151f, 0.00152f, 0.00153f, 0.00151f, 0.00152f, 0.00152f, 0.00148f,
  0.0015f, 0.00148f, 0.00149f, 0.00147f, 0.00147f, 0.00146f, 0.00144f, 0.
  00145f, 0.00144f, 0.00143f, 0.00143f, 0.0014f, 0.0014f, 0.00139f, 0.00138f,
  0.00138f, 0.00138f, 0.00137f, 0.00138f, 0.00136f, 0.00134f, 0.00134f, 0.
  00133f, 0.00132f, 0.00131f, 0.00128f, 0.00129f, 0.00126f, 0.00126f, 0.00127f,
  0.00124f, 0.00122f, 0.00121f, 0.0012f, 0.0012f, 0.00119f, 0.00118f, 0.00116f,
  0.00117f, 0.00111f, 0.00115f, 0.00109f, 0.0011f, 0.00109f, 0.00107f, 0.
  00106f, 0.00105f, 0.00103f, 0.00102f, 0.001f, 0.00099f, 0.00099f, 0.00096f,
  0.00095f, 0.00093f, 0.00092f, 0.00093f, 0.0009f, 0.00087f, 0.00087f, 0.
  00083f, 0.00083f, 0.00081f, 0.00079f, 0.00078f, 0.00078f, 0.00074f, 0.00073f,
  0.00071f, 0.00069f, 0.00069f, 0.00067f, 0.00064f, 0.00063f, 0.00061f, 0.
  00061f, 0.00058f, 0.00057f, 0.00053f, 0.00053f, 0.0005f, 0.00049f, 0.00048f,
  0.00046f, 0.00045f, 0.00044f, 0.00042f, 0.00041f, 0.00038f, 0.00038f, 0.
  00036f, 0.00033f, 0.00033f, 0.0003f, 0.00028f, 0.00027f, 0.00025f, 0.00025f,
  0.00022f, 0.00022f, 0.00019f, 0.00018f, 0.00015f, 0.00014f, 0.00013f, 0.
  00012f, 0.0001f, 0.00009f, 0.00007f, 0.00003f, 0.00005f, 0.00002f, 0.00001f,
  0.0f, -0.00001f, -0.00003f, -0.00005f, -0.00007f, -0.00009f, -0.0001f, -0.
  00012f, -0.00014f, -0.00015f, -0.00018f, -0.00019f, -0.0002f, -0.00022f, -0.
  00022f, -0.00025f, -0.00027f, -0.00028f, -0.0003f, -0.00031f, -0.00033f, -0.
  00034f, -0.00036f, -0.00037f, -0.00037f, -0.00039f, -0.00041f, -0.00041f, -0.
  00045f, -0.00045f, -0.00047f, -0.00049f, -0.00049f, -0.00053f, -0.00051f, -0.
  00057f, -0.00054f, -0.00058f, -0.00057f, -0.00059f, -0.0006f, -0.00062f, -0.
  00064f, -0.00064f, -0.00066f, -0.00066f, -0.00067f, -0.00071f, -0.00069f, -0.
  00073f, -0.00071f, -0.00073f, -0.00073f, -0.00074f, -0.00075f, -0.00074f, -0.
  00075f, -0.00078f, -0.00077f, -0.0008f }

===============================================================================
test cases:   545 |   544 passed | 1 failed
assertions: 49637 | 49636 passed | 1 failed

@redtide redtide added the ci continuous integration related label Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working ci continuous integration related
Projects
None yet
Milestone
1.3.0
Development

No branches or pull requests
2 participants
@redtide @paulfd