-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Disable auto-download by default + improve test stability #3076
Changes from all commits
df5f5f4
7ba40bd
e550343
c479e0d
26d03f0
329d461
68cb6ee
8269633
0dadc81
1abba7e
bc57be3
fa2f849
8932e48
7838ef5
14dd23f
d9d3122
af9d529
f3f8855
5eb6405
05dbdc2
8c6e1ea
e0d930e
248c72d
440cffb
62bb419
42ae6f2
a9a8e0c
c6cc8e7
3a5d81b
5b31dd3
3f745ad
a317180
2e25c69
5de5a6e
cf3df9b
55f2e5f
c624159
605d315
a276c72
a2eb66c
2a2938b
fd10d8c
054c23f
8844a3a
65bce43
0639a47
69000ac
1e5ee4b
f063f13
5ac9c90
af90372
7359cae
bf71db3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,15 @@ steps: | |
|
||
# Install build dependencies. | ||
- script: | | ||
# Workaround to avoid a build failure, because we run out of memory. | ||
free -h | ||
df -h | ||
sudo dd if=/dev/zero of=/usr/swapfile.img bs=1024 count=12M | ||
sudo mkswap /usr/swapfile.img | ||
sudo swapon /usr/swapfile.img | ||
free -h | ||
df -h | ||
|
||
git clone --depth 1 https://github.com/mlpack/jenkins-conf.git conf | ||
|
||
sudo add-apt-repository ppa:mhier/libboost-latest | ||
|
@@ -30,12 +39,12 @@ steps: | |
fi | ||
|
||
if [ "a$(julia.version)" != "a" ]; then | ||
wget https://julialang-s3.julialang.org/bin/linux/x64/1.3/julia-1.3.0-linux-x86_64.tar.gz | ||
sudo tar -C /opt/ -xvpf julia-1.3.0-linux-x86_64.tar.gz | ||
wget https://julialang-s3.julialang.org/bin/linux/x64/1.6/julia-1.6.3-linux-x86_64.tar.gz | ||
sudo tar -C /opt/ -xvpf julia-1.6.3-linux-x86_64.tar.gz | ||
fi | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I can't provide it as a suggestion, but I think simply changing the URL should be sufficient:
|
||
|
||
# Install armadillo. | ||
curl -L https://sourceforge.net/projects/arma/files/armadillo-9.800.6.tar.xz | tar -xvJ && \ | ||
curl -k -L https://sourceforge.net/projects/arma/files/armadillo-9.800.6.tar.xz | tar -xvJ && \ | ||
cd armadillo* && \ | ||
cmake . && \ | ||
make && \ | ||
|
@@ -45,7 +54,7 @@ steps: | |
# Install ensmallen. | ||
wget https://ensmallen.org/files/ensmallen-latest.tar.gz | ||
tar -xvzpf ensmallen-latest.tar.gz # Unpack into ensmallen-*/. | ||
cd ensmallen-* && \ | ||
cd ensmallen-*/ && \ | ||
sudo cp -vr include/* /usr/include/ && \ | ||
cd .. | ||
|
||
|
@@ -77,13 +86,7 @@ steps: | |
displayName: 'CMake' | ||
|
||
# Build mlpack | ||
- script: cd build && make && make mlpack_test | ||
condition: eq(variables['CMakeArgs'], '-DDEBUG=ON -DPROFILE=OFF -DBUILD_PYTHON_BINDINGS=OFF -DBUILD_JULIA_BINDINGS=OFF -DBUILD_GO_BINDINGS=OFF -DBUILD_R_BINDINGS=OFF') | ||
displayName: 'Build' | ||
|
||
# Build mlpack | ||
- script: cd build && make -j2 && make -j2 mlpack_test | ||
condition: ne(variables['CMakeArgs'], '-DDEBUG=ON -DPROFILE=OFF -DBUILD_PYTHON_BINDINGS=OFF -DBUILD_JULIA_BINDINGS=OFF -DBUILD_GO_BINDINGS=OFF -DBUILD_R_BINDINGS=OFF') | ||
- script: cd build && make -j2 && make mlpack_test -j2 | ||
displayName: 'Build' | ||
|
||
# Run tests via ctest. | ||
|
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -51,9 +51,7 @@ jobs: | |||||
- name: Install Build Dependencies | ||||||
run: | | ||||||
sudo apt-get update | ||||||
sudo apt-get install -y --allow-unauthenticated libopenblas-dev liblapack-dev g++ libboost-all-dev libcereal-dev libcurl4-openssl-dev | ||||||
curl -L https://sourceforge.net/projects/arma/files/armadillo-9.800.6.tar.xz | tar -xvJ && cd armadillo* | ||||||
cmake . && make && sudo make install && cd .. | ||||||
sudo apt-get install -y --allow-unauthenticated libopenblas-dev liblapack-dev g++ libboost-all-dev libcereal-dev libensmallen-dev libhdf5-dev libarmadillo-dev libcurl4-openssl-dev | ||||||
|
||||||
- name: Install R-bindings dependencies | ||||||
run: | | ||||||
|
@@ -64,7 +62,7 @@ jobs: | |||||
- name: CMake | ||||||
run: | | ||||||
mkdir build | ||||||
cd build && cmake -DDEBUG=OFF -DPROFILE=OFF -DBUILD_CLI_EXECUTABLES=OFF -DBUILD_PYTHON_BINDINGS=OFF -DBUILD_JULIA_BINDINGS=OFF -DBUILD_GO_BINDINGS=OFF -DBUILD_R_BINDINGS=ON .. | ||||||
cd build && cmake -DDEBUG=OFF -DPROFILE=OFF -DBUILD_CLI_EXECUTABLES=OFF -DBUILD_PYTHON_BINDINGS=OFF -DBUILD_JULIA_BINDINGS=OFF -DBUILD_GO_BINDINGS=OFF -DBUILD_R_BINDINGS=ON -DDOWNLOAD_DEPENDENCIES=ON .. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
I read the log but don't really understand what happened. Can you try this? And maybe also add There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Forgot to remove the previously used armadillo build command. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess I didn't look high enough up in the log :) |
||||||
|
||||||
- name: Build | ||||||
run: | | ||||||
|
@@ -127,6 +125,12 @@ jobs: | |||||
key: ${{ runner.os }}-r-${{ matrix.config.r }}-${{ hashFiles('depends.Rds') }} | ||||||
restore-keys: ${{ runner.os }}-r-${{ matrix.config.r }}- | ||||||
|
||||||
- name: Install check dependencies | ||||||
if: runner.os != 'Windows' && runner.os != 'macOS' | ||||||
run: | | ||||||
sudo apt-get update | ||||||
sudo apt-get install -y --allow-unauthenticated libcurl4-openssl-dev | ||||||
|
||||||
- name: Install dependencies | ||||||
run: | | ||||||
install.packages('remotes') | ||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -162,12 +162,12 @@ Convolution< | |
padWRight(layer.padWRight), | ||
padHBottom(layer.padHBottom), | ||
padHTop(layer.padHTop), | ||
weights(layer.weights), | ||
inputWidth(layer.inputWidth), | ||
inputHeight(layer.inputHeight), | ||
outputWidth(layer.outputWidth), | ||
outputHeight(layer.outputHeight), | ||
padding(layer.padding), | ||
weights(layer.weights) | ||
padding(layer.padding) | ||
{ | ||
// Nothing to do here. | ||
} | ||
|
@@ -197,12 +197,12 @@ Convolution< | |
padWRight(layer.padWRight), | ||
padHBottom(layer.padHBottom), | ||
padHTop(layer.padHTop), | ||
weights(std::move(layer.weights)), | ||
inputWidth(layer.inputWidth), | ||
inputHeight(layer.inputHeight), | ||
outputWidth(layer.outputWidth), | ||
outputHeight(layer.outputHeight), | ||
padding(std::move(layer.padding)), | ||
weights(std::move(layer.weights)) | ||
padding(std::move(layer.padding)) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you for these warning fixes! 💯 |
||
{ | ||
// Nothing to do here. | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll also need to change the Julia versions here to
1.6.3
(throughout the file), and the-DJULIA_EXECUTABLE
option to-DJULIA_EXECUTABLE=/opt/julia-1.6.3/bin/julia
.