CompEvol / beast2

Bayesian Evolutionary Analysis by Sampling Trees
www.beast2.org
GNU Lesser General Public License v2.1
240 stars 84 forks source link

RPNcalculator is fun but awkward to use for non-HP calculator fans #81

Closed tgvaughan closed 10 years ago

tgvaughan commented 10 years ago

Really nice parser generators abound these days - we could probably produce something more accessible in an hour or two.

rbouckaert commented 10 years ago

I suggest to move this functionality to BEAST labs.

Adding many classes that do not contain core functionality should not be in the core. Adding jars to the core is frowned upon.

On Thu, 2014-03-20 at 02:09 -0700, Tim Vaughan wrote:

Closed #81 via 38dae86.

— Reply to this email directly or view it on GitHub.

tgvaughan commented 10 years ago

I tried to keep those classes tucked away in their own little package - they're just generated by ANTLR. I also thought that ANTLR would probably be of use in cleaning up some other parsers in the core. Fair enough, though. Reverted in 60b3f317b8d6b6fb265d52bb67fae9eef05056e4.

On 21 March 2014 06:50, rbouckaert notifications@github.com wrote:

I suggest to move this functionality to BEAST labs.

Adding many classes that do not contain core functionality should not be in the core. Adding jars to the core is frowned upon.

On Thu, 2014-03-20 at 02:09 -0700, Tim Vaughan wrote:

Closed #81 via 38dae86.

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/CompEvol/beast2/issues/81#issuecomment-38199231 .