Add cmt

[jakirkham]

Apparently my description got wiped out when I moved it on ZenHub boards. So, I'm re-writing based on what I remember. 😢

We have tried to package cmt. This is a brand new recipe. A number of issues were found with the latest release. These have all been submitted upstream. All patches that made sense have been submitted upstream. There is one workaround patch (compiler change for Mac), which has not been submitted upstream. However, an issue has been opened to discuss changes that would make it unnecessary. Below is a list of either PRs (with issues referenced) or issues (if no PR was submitted). Also, a list of otherwise related issues has been compiled below.

Patches:

• Respect CC, CXX, etc. from environment if provided lucastheis/cmt#10
• Fix issue 14: "Test failure on 0.5.0 (due to missing import)" lucastheis/cmt#18
• Fix issue 12: "Test failure on 0.5.0 (due to pickling failure)" lucastheis/cmt#17
• Fix issue 16: "Another test failure on 0.5.0 (due to sporadic numerical error?)" lucastheis/cmt#20
• Fix issue 15: "Test failure on 0.5.0 (due to numerical error?)" lucastheis/cmt#19

Related issues:

• New release? lucastheis/cmt#8
• Using a BLAS other than MKL lucastheis/cmt#9
• Using an external copy of Eigen lucastheis/cmt#11
• Add SciPy as a dependency lucastheis/cmt#13
• CMake build for lbfgs lucastheis/cmt#21

[conda-forge-admin]

Hi! This is the friendly conda-forge-admin automated user.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/cmt) and found it was in an excellent condition.

[jakirkham]

It seems some of the tests may be too intense for Circle CI. We may need to skip some of them on that platform.

[jakirkham]

Anybody have feedback here? Personally, I am ok with this, but maybe there are things that could be improved.

[jakirkham]

@ocefpaf, thoughts?

If nothing too serious, I am thinking I will merge this once CI passes. It takes awhile to build and test so I don't want to cut into the build queue time at staged-recipes.

[jakirkham]

Feedback welcome at the feedstock.

[ocefpaf]

👍

[jakirkham]

Sorry, was pretty fast on merging.

Glad to hear you approve though. 😄

[jakirkham]

@lucastheis, we are adding cmt to conda-forge. This is the community packaging system for use with conda. Packages built here are built with CI and deployed to the conda-forge channel automatically after committing on master of a feedstock (a repo with the recipe plus automatically maintained CI scripts). Eventually, packages submitted here will be included in the defaults channel after some auditing process as mentioned. If there is anything here that you see is off or needs to be fixed, please let me know. If you would like to have access to the cmt feedstock/recipe, I would happily add you to the maintainers list so you can have commit privileges for this recipe. This would be useful for updating the version for instance. Thanks again for working on such a useful library.

As a side note, there were a few testing issues I ran into, which I have filed issues for. Also, all the patches I have here have been submitted as PRs. Once these are resolved, it would be nice to see a fresh release on master so we can get Python 3.x support and fixes to these issues.