febio: 3.6 -> 4.7; febio-studio: 1.6.1 -> 2.7#331864
febio: 3.6 -> 4.7; febio-studio: 1.6.1 -> 2.7#331864philiptaron merged 7 commits intoNixOS:masterfrom
Conversation
ofborg
bot
added
10.rebuild-darwin: 1-10
philiptaron
left a comment
philiptaron
left a comment
There was a problem hiding this comment.
Gave a thorough build check.
|
|
||
| patches = [ | ||
| # Fix library searching and installation | ||
| (substituteAll { |
There was a problem hiding this comment.
Thanks for the pointers! #301350 looks interesting. This is just inherited from the previous version of the derivation, though.
There was a problem hiding this comment.
Yeah, hence "unrelated" -- no expectation for you to take any action.
| ninja, | ||
| zlib, | ||
| darwin, | ||
| mklSupport ? true, |
There was a problem hiding this comment.
Tested with mklSupport off on x86_64-linux; builds.
I know it's in the previous version, but this does make febio a mixed free/non-free derivation.
There was a problem hiding this comment.
Yeah, I get the impression that you’re really just meant to use it:
FEBio requires the Intel Math Kernel Library (MKL) in order to utilize the Pardiso linear solver and some of the iterative linear solvers. This library can be downloaded as part of the Intel oneAPI Base Toolkit from Intel's website: https://software.intel.com/content/www/us/en/develop/tools/oneapi/base-toolkit.html. In the absence of MKL, FEBio will default to using the Skyline linear solver. However, the Pardiso solver is significantly faster and more memory-efficient than the Skyline solver, and it is strongly recommended that the Pardiso solver be used.
| zlib, | ||
| libGLU, | ||
| qt6Packages, | ||
| febio, |
There was a problem hiding this comment.
Confirmed that this derivation builds with febio = febio.override { mklSupport = false; }
| dicomSupport ? false, | ||
| dcmtk, | ||
| withModelRepo ? true, | ||
| withCadFeatures ? false, |
There was a problem hiding this comment.
confirmed that this builds with withCadFeatures set to true
| tetgen, | ||
| ffmpegSupport ? true, | ||
| ffmpeg_7, | ||
| dicomSupport ? false, |
There was a problem hiding this comment.
confirmed that this builds with dicomSupport set to true
| libssh, | ||
| tetgenSupport ? true, | ||
| tetgen, | ||
| ffmpegSupport ? true, |
There was a problem hiding this comment.
confirmed that this builds with ffmpegSupport set to false
| sshSupport ? true, | ||
| openssl, | ||
| libssh, | ||
| tetgenSupport ? true, |
There was a problem hiding this comment.
confirmed that this builds with tetgenSupport set to false
| ffmpeg_7, | ||
| dicomSupport ? false, | ||
| dcmtk, | ||
| withModelRepo ? true, |
There was a problem hiding this comment.
confirmed that this builds with withModelRepo set to false
|
I intend to merge on Monday, in order to give @Scriptkiddi time to look. But this seems 👍🏻 |
|
thanks for taking care of this, im not using febio really active anymore but I know its being used for research by 8 people |
3 participants
Description of changes
Phew, this was a lot of work! Another FFmpeg 4 dependency upgraded, and some Darwin support to sweeten the deal.
@Scriptkiddi I hope you still use this! I have absolutely no idea how to operate the program, but I’ve checked that it starts and can call out to FEBio and so on. It would be great if you could test it more thoroughly, especially on Linux.
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.