[mlpack-git] [mlpack/mlpack] Default to DEBUG=OFF and PROFILE=OFF. (#662)

Sterling Peet notifications at github.com
Sat Jun 4 11:54:03 EDT 2016


Using master for mainline development (or not) is an interesting issue.  Doing so is closely aligned with the SVN 'trunk' development and branching model, but may or may not be appropriate in git.

I have noticed that git based projects (especially on GitHub) present a much more professional look and feel when using master as the stable branch, in no small part because GitHub presents the master branch as the default landing page.  So, perhaps it might be worth considering the master branch in light of it being the first public introduction for someone interested in trying out mlpack.

For unrelated reasons, my personal development evolved to follow the [Successful Git Branching Model](http://nvie.com/posts/a-successful-git-branching-model/) proposed/documented by [Vincent Driessen](http://nvie.com/about/).  This model uses master as the branch that holds the stable, release code only.  Release tags in this model all point into the master branch somewhere.

That model clearly doesn't address the concerns of making experimental features available to the end user, but that may not be a bad thing.  Personally, I feel that the need for unavailable experimental features indicates that surpassing a higher barrier to entry (really just the ability to checkout a different git branch in this case) is not unreasonable to ask, and my professionally developed projects reflect that philosophy.

In any case, I think the readme in the master branch should have and indication of what branch of code a user wants for normal use cases, like benchmarking or advanced (or unreleased) features.

Hopefully these comments are useful for considering how to mitigate the root concern, dealing with the unexpected things people do is challenging.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/662#issuecomment-223762927
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160604/582ec226/attachment.html>


More information about the mlpack-git mailing list