Skip to content

Conversation

ShuangLiu1992
Copy link
Contributor

ShuangLiu1992 added a commit to ShuangLiu1992/gtsam that referenced this pull request Jan 27, 2023
@dellaert
Copy link
Member

Thanks ! Let's wait for CI.
Is this also an issue in release/4.2 or is this due to changes in develop after we branched off the release?

@dellaert dellaert merged commit 7bf72c3 into borglab:develop Jan 28, 2023
@dellaert
Copy link
Member

@ShuangLiu1992 I merged but still interested in answer about 4.2. Thanks for your contribution, regardless!

@ShuangLiu1992
Copy link
Contributor Author

Hi thanks for merging.
Previously I have been only compiling release 4.2a7 and it was fine on all compilers I tested, recently started trying out the develop branch and notcied clang reporting this error. I haven't tried 4.2a8

@varunagrawal
Copy link
Contributor

I'm on 4.2 with clang and haven't received this error. We should still wait for @ShuangLiu1992 to report his results though.

@dellaert
Copy link
Member

Actually, both are new methods added by Kartik and are not even in 4.2, so we can ignore. Thanks both!

@ShuangLiu1992 ShuangLiu1992 deleted the patch-1 branch January 30, 2023 13:45
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.

3 participants