-
Notifications
You must be signed in to change notification settings - Fork 19
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
build: Add QuickPlot into QML GUI (#1963)
- Loading branch information
Showing
13 changed files
with
157 additions
and
24 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -45,10 +45,45 @@ runs: | |
run: | | ||
set -ex | ||
### Begin unholy hack | ||
### | ||
### For reasons unknown, when nix is run under GitHub actions, | ||
### it ignores the submodules flag, meaning that flakes with | ||
### submodules cannot be built | ||
### | ||
### As a blasphemous workaround, I've added a step to bring the | ||
### submodule into the main repo. Hopefully, submodules are | ||
### made the default flake behaviour in the upcoming release, | ||
### this will no longer be necessary. If not, we might have to | ||
### make the submodule an extra input to the flake, but that | ||
### will take some extra time. | ||
### | ||
### If you are reading this comment in 2026, try removing this | ||
### entire hack and see if the problem fixed itself. You may | ||
### also need to update the action that installs nix in the | ||
### first place. If that doesn't solve the problem, then we | ||
### should go ahead and just make the submodule a flake input. | ||
git submodule update --init --recursive | ||
git config --global user.email "[email protected]" | ||
git config --global user.name "GitHub Actions" | ||
git rm .gitmodules | ||
cp -r QuickPlot QuickPlot.bak | ||
rm -rf QuickPlot.bak/.git | ||
git rm -r QuickPlot | ||
git commit -m "Kill submodule" | ||
mv QuickPlot.bak QuickPlot | ||
git add QuickPlot | ||
git commit -m "Resurrect submodule" | ||
### End unholy hack | ||
# Build Singularity target | ||
target=${{ inputs.target }} | ||
singularityTarget=${target/dissolve/singularity} | ||
nix build -L .#$singularityTarget | ||
nix build -L ".?submodules=1#$singularityTarget" | ||
# Assemble artifacts | ||
mkdir packages-sif | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
[submodule "QuickPlot"] | ||
path = QuickPlot | ||
url = https://github.com/disorderedmaterials/QuickPlot.git |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
f48d284
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.
Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold
2
.BM_HistogramBinning_3d/16777216
57.640470536140455
ns/iter28.180865817928062
ns/iter2.05
This comment was automatically generated by workflow using github-action-benchmark.
CC: @disorderedmaterials/dissolve-devs