search

mmaul / clml

Common Lisp Machine Learning Library
Other
259 stars 36 forks source link

Cannot load with latest SBCL #29

Closed anquegi closed 6 years ago

anquegi commented 6 years ago

loading clml with latest quicklisp and latest clml, gives the following error:

CL-USER> (ql:quickload :clml)
To load "clml":
  Load 1 ASDF system:
    clml
; Loading "clml"
.......

; file: /Users/toni/.roswell/lisp/quicklisp/dists/quicklisp/software/clml-20170630-git/time-series/src/ts-anomaly-detection.lisp
; in: DEFGENERIC E-SCORES
;     (T CLML.TIME-SERIES.ANOMALY-DETECTION::R)
; 
; caught ERROR:
;   (during macroexpansion of (DEFGENERIC E-SCORES
;       ...))
;   COMMON-LISP:T names a defined constant, and cannot be used in
;   a generic function lambda list.
;   See also:
;     The ANSI Standard, Section 3.4.2
........; 
        ; compilation unit aborted
        ;   caught 2 fatal ERROR conditions
        ;   caught 1 ERROR condition
; Evaluation aborted on #<UIOP/LISP-BUILD:COMPILE-FILE-ERROR {1009CB7913}>.

i'm using:

CL-USER> (lisp-implementation-type)
"SBCL"
CL-USER> (lisp-implementation-version)
"1.3.20"

it seems due to this update

minor incompatible change: DEF{GENERIC,METHOD} no longer accept some illegal lambda lists such as (defgeneric bar (foo &key foo)) or (defgeneric baz (t)) that were accepted before.

anquegi commented 6 years ago

Is the quicklisp version on new releases it works, forked the repo an load as local-project, and it works so this commit owas a week ago corrected this, CL-USER> (ql:update-all-dists) 2 dists to check. Downloading http://beta.quicklisp.org/dist/quicklisp.txt ########################################################################## You already have the latest version of "quicklisp": 2017-07-25.. iT was just corrected so in new version of quicklisp it should work for now in 

CL-USER> (ql:update-all-dists)
2 dists to check.
Downloading http://beta.quicklisp.org/dist/quicklisp.txt
##########################################################################
You already have the latest version of "quicklisp": 2017-07-25.

so for next dist should work, forgiveme for opening this issue

mmaul commented 6 years ago

Thanks for reporting, and glad it didn't inconvenience you for long :)