-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add recipe for blasfeo #27207
Add recipe for blasfeo #27207
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/blasfeo/meta.yaml:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
b465c18
to
a5c4ad1
Compare
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/blasfeo/meta.yaml:
|
a5c4ad1
to
693a231
Compare
@conda-forge/staged-recipes @conda-forge/help-c-cpp the PR is ready for review. The linter complains "Please do not delete the example recipe found in recipes/example/recipe.yaml.", but as the file has not been touched by this PR and the same error occurs on unrelated PRs (see #27234 or #27235), I guess the problem may be in the linter? |
To help direct your pull request to the best reviewers, please mention a topic-specifc team if your recipe matches any of the following: conda-forge/help-c-cpp, conda-forge/help-cdts, conda-forge/help-go, conda-forge/help-java, conda-forge/help-julia, conda-forge/help-nodejs, conda-forge/help-perl, conda-forge/help-python, conda-forge/help-python-c, conda-forge/help-r, conda-forge/help-ruby,or conda-forge/help-rust. Thanks! |
Not sure what is going on with the linter there. The example recipe is in there :-/ |
blasfeo provides a set of basic linear algebra routines, performance-optimized for matrices that fit in cache (i.e. generally up to a couple hundred size in each dimension), as typically encountered in embedded optimization applications.
The package name (
libblasfeo
) contains thelib
prefix as requested in #19764 (review) for C/C++ libraries. The feedstock is namedblasfeo
to match the repo name and support different package outputs in the future, if that turns out to be necessary.I am adding blasfeo as it is a transitive dependency for a new version of the casadi library, see conda-forge/casadi-feedstock#112 .
The latest tagged version of blasfeo is relatively old and does not compile on macOS and Windows. I asked upstream if they are interested in tagging a new release (see giaf/blasfeo#181), but in the meanwhile I think we can start packaging (just for Linux) the latest tagged version.
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).