Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

TallJimbo
Copy link
Member

@TallJimbo TallJimbo commented Jul 29, 2019

Previous PR (#76) caused a Travis failure after merge to master and was quickly reverted, which is quite strange considering that Travis passed on the branch.

@TallJimbo
Copy link
Member Author

At least some of the build problems here were pre-existing; master fails in Travis too if I kick it.

Debugging that has at least reminded me of the issue with Eigen 3.2: there is support for equivalent functionality with Eigen 3.3 via a different interface, and I just didn't want to remove the old one completely without some deprecation period, so it's been gated in CMake. Over on #79 I'm now removing it entirely, along with a bunch of other cruft that's unused by LSST or (AFAIK) any other ndarray users. Once that's done I'll either try to cherry-pick some of your commits or ask you to rebase and clean up your branch, depending on how much time I have.

@TallJimbo
Copy link
Member Author

Closing in favor of #80, which is my rebased version of this branch without the need to pin Eigen.

@TallJimbo TallJimbo closed this Jul 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants