Skip to content

Undefine the WE_LOVE_228 macro and fix the tests #1608

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

Merged
merged 1 commit into from
Oct 15, 2017

Conversation

pleroy
Copy link
Member

@pleroy pleroy commented Oct 15, 2017

No description provided.

@eggrobin
Copy link
Member

eggrobin commented Oct 15, 2017

Note: this is a first step towards resolving #228, to get sample dense trajectories to investigate potential downsampling methods.

Sorry, something went wrong.

@@ -229,9 +229,5 @@ template_and_class_key declared_name; \
} \
using internal_##package_name::declared_name

// We preserve issue #228 in Bourbaki because we don't have trajectory
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, Bourbaki :D

Sorry, something went wrong.

@eggrobin eggrobin added the LGTM label Oct 15, 2017
@pleroy pleroy merged commit 44b8c11 into mockingbirdnest:master Oct 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants