Skip to content

Conversation

@bmschmidt
Copy link

This is a dummy request in that I don't think you should merge it: I just wanted to ping and say hey, here's the code it takes to build a Bookworm on the type of feature data you're sharing.

Copy link
Owner

@tedunderwood tedunderwood left a comment

Choose a reason for hiding this comment

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

Thanks! This generalization is much appreciated.

@bmschmidt
Copy link
Author

bmschmidt commented Apr 29, 2018

Oh oops, you should actually roll most of this back: I thought I was pushing to a local fork of mine, not yours. There might be a piece or two you want, but most of this is just to allow a replication using a different feature set for a paper that probably doesn't make sense.

@bmschmidt
Copy link
Author

Sorry for the confusion!

@tedunderwood
Copy link
Owner

No problem. I don't think I actually merged anything. I just "reviewed the changes" and "approved them" -- but that doesn't do anything to the branch as far as I can tell!

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