search

cardmagic / classifier

A general classifier module to allow Bayesian and LSI classifications.
Other
661 stars 122 forks source link

require 'mathn' breaks calculations #10

Closed christianhellsten closed 4 months ago

christianhellsten commented 13 years ago

require 'mathn' breaks calculations:

(10 / 23).class
=> Fixnum
require 'classifier'
(10 / 23).class
=> Rational

This means e.g. date calculations will break:

Date.civil(year, month, day)
=> ArgumentError: invalid date

See: http://blog.brightredglow.com/2008/1/17/evil-can-be-dangerous/

jimknight commented 12 years ago

Also, this breaks Chronic.parse...

example: Chronic.parse("01/13/11") produces 1973-01-13

found that installing gsl gem overrides this route which 'solves' the issue

cardmagic commented 4 months ago

I've removed mathn thank you