-
-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support bb/sci #59
Comments
https://github.com/wkok/openai-clojure/blob/main/src/wkok/openai_clojure/sse.clj#L10-L12 Since babashka doesn't support loading arbitrary java classes: https://book.babashka.org/#libraries, the way it imports a LinkedList will not work in babashka. |
The usage of I have not had a chance to look into it but you're welcome to raise a PR that either
|
Fixed by #60 |
So, I'm using the library , so far so good. However when I try to use it in babashka script, I'm getting an error saying LinkedList is not supported in bb:
The
bb.edn
file:The script:
Error message:
bb tst.clj ----- Error -------------------------------------------------------------------- Type: java.lang.Exception Message: Unable to resolve classname: java.util.LinkedList Location: wkok/openai_clojure/sse.clj:10:3 ----- Context ------------------------------------------------------------------ 6: [clojure.core.async :as a] 7: [clojure.string :as string] 8: [cheshire.core :as json] 9: [clojure.core.async.impl.protocols :as impl]) 10: (:import (java.io InputStream) ^--- Unable to resolve classname: java.util.LinkedList 11: (clojure.lang Counted) 12: (java.util LinkedList))) 13: 14: (def event-mask (re-pattern (str "(?s).+?\n\n"))) 15: ----- Stack trace -------------------------------------------------------------- wkok.openai-clojure.sse - wkok/openai_clojure/sse.clj:10:3 wkok.openai-clojure.azure - wkok/openai_clojure/azure.clj:2:3 wkok.openai-clojure.core - wkok/openai_clojure/core.clj:2:3 wkok.openai-clojure.api - wkok/openai_clojure/api.clj:2:3 user - /Users/jianghongying/code/conavi/tst.clj:1:1
Any thoughts on adding support to it? I believe the culprit must be how the LinkedList is used the first place.
The text was updated successfully, but these errors were encountered: