Skip to content
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

Get Duckling to work #143

Open
argideritzalpea opened this issue May 22, 2019 · 9 comments · Fixed by #144
Open

Get Duckling to work #143

argideritzalpea opened this issue May 22, 2019 · 9 comments · Fixed by #144

Comments

@argideritzalpea
Copy link

I am attempting to incorporate Duckling project with eta but am running into some dependency issues. My main goal is to eventually use Duckling (Haskell) in an Android application. I am first attempting to get Duckling running via eta.

Duckling: https://github.com/facebook/duckling
It is also on Hackage: https://hackage.haskell.org/package/duckling
I clone the Duckling gitHub project here, and I run:
home$ etlas run

Resolving dependencies...
etlas: Could not resolve dependencies:
trying: base-4.11.1.0/installed-9g5... (dependency of duckling-0.1.6.1)
next goal: text-show (dependency of duckling-0.1.6.1)
rejecting: text-show-3.8.2, text-show-3.8.1, text-show-3.8 (conflict: duckling
=> text-show>=2.1.2 && <3.8)
trying: text-show-3.7.5
next goal: integer-gmp (dependency of text-show-3.7.5)
rejecting: integer-gmp-1.0.2.0 (conflict: base =>
ghc-prim==0.4.0.0/installed-Jhi..., integer-gmp => ghc-prim>=0.5.2)
rejecting: integer-gmp-1.0.1.0 (conflict: base =>
ghc-prim==0.4.0.0/installed-Jhi..., integer-gmp => ghc-prim>=0.5.1)
rejecting: integer-gmp-1.0.0.1 (conflict: base =>
ghc-prim==0.4.0.0/installed-Jhi..., integer-gmp => ghc-prim>=0.5)
rejecting: integer-gmp-1.0.0.0, integer-gmp-0.5.1.0 (only already installed
instances can be used)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: integer-gmp, base, text-show, duckling

@jneira
Copy link
Collaborator

jneira commented May 22, 2019

Hi @argideritzalpea, some libs needed to be patched to make it work: #144
When the pr is merged it should be buildable with etlas update && etlas build

@rahulmutt
Copy link
Member

@argideritzalpea @jneira PR is merged.

@argideritzalpea
Copy link
Author

@jneira @rahulmutt Thanks very much for the patches and PR. I'm having trouble building it still after updating etlas:

$ etlas update && etlas build
Downloading the latest package lists from:
- hackage.haskell.org
- git@github.com/typelead/etlas-index
Updating binary package index.
To revert to previous state run:
    etlas update --index-state='2019-05-23T17:03:17Z'
Updating the eta-hackage patch set.
Resolving dependencies...
etlas: Could not resolve dependencies:
trying: base-4.11.1.0/installed-9g5... (dependency of duckling-0.1.6.1)
next goal: text-show (dependency of duckling-0.1.6.1)
rejecting: text-show-3.8.2, text-show-3.8.1, text-show-3.8 (conflict: duckling
=> text-show>=2.1.2 && <3.8)
trying: text-show-3.7.5
next goal: integer-gmp (dependency of text-show-3.7.5)
rejecting: integer-gmp-1.0.2.0 (conflict: base =>
ghc-prim==0.4.0.0/installed-Jhi..., integer-gmp => ghc-prim>=0.5.2)
rejecting: integer-gmp-1.0.1.0 (conflict: base =>
ghc-prim==0.4.0.0/installed-Jhi..., integer-gmp => ghc-prim>=0.5.1)
rejecting: integer-gmp-1.0.0.1 (conflict: base =>
ghc-prim==0.4.0.0/installed-Jhi..., integer-gmp => ghc-prim>=0.5)
rejecting: integer-gmp-1.0.0.0, integer-gmp-0.5.1.0 (only already installed
instances can be used)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: integer-gmp, base, text-show, duckling

@jneira
Copy link
Collaborator

jneira commented May 23, 2019

Oh, i forgot to mention that you should download the package with etlas get duckling-0.1.6.1 again to get the version patched. When downloading it should output Found patch in eta-hackage for duckling-0.1.6.1

@argideritzalpea
Copy link
Author

@jneira @rahulmutt Thanks for helping me get this running, it successfully built! I can debug some of the runtime errors now

@jneira
Copy link
Collaborator

jneira commented May 29, 2019

As noted by @argideritzalpea, after compiling there are two issues:

  • Being able to run the web app example that uses snap
  • Run the test suite

@jneira jneira reopened this May 29, 2019
@jneira jneira pinned this issue May 29, 2019
@jneira
Copy link
Collaborator

jneira commented May 30, 2019

Hi, i tried to run the test suite and, at first, i've got a compile error, solved in the new patch: #146

I actually i am getting a java exception instead the error about WrapPCRE.hsc described in #144 (comment).

PS D:\dev\lang\eta\eta-pkgs\duckling-0.1.6.1> .\dist\build\eta-0.8.6.5\duckling-0.1.6.1\t\d
uckling-test\build\duckling-test\duckling-test.cmd
Duckling Tests
  API Tests
    Parse Test:                    Exception in thread "Thread-9" Exception in thread "Thread-8" Exception in thread "Thread-7" java.lang.NoClassDefFoundError: regex_pcre_bu__0.94.4.__3bceb134cf8075d23ca76fbd3ae999e26aae2df4/text/regex/pcre/ByteString$$w$cmatchAll
        at duckling.duckling.Engine$$wlookupRegexAnywhere.call(Engine.hs:94)
        at duckling.duckling.Engine$go3.apply1(Engine.hs:187)
        at duckling.duckling.Engine$sat$29.thunkEnter(Engine.hs:187)
        at eta.runtime.thunk.SingleEntryThunk.evaluate(SingleEntryThunk.java:19)
        at eta.runtime.stg.Closure.evaluateTail(Closure.java:29)
        at base.ghc.Base$zpzp.call(Base.hs:1145)
        at duckling.duckling.Engine$$wparseString1.call(Engine.hs:181)
        at duckling.duckling.Engine$parseAndResolve.call(Engine.hs:211)
        at duckling.duckling.Api$$wanalyzze.call(Api.hs:56)
        at duckling.duckling.Api$$wparse.call(Api.hs:40)
        at main.duckling.api.Tests$tests147.call(Tests.hs:52)
        at main.duckling.api.Tests$tests147.applyV(Tests.hs)
....

@rahulmutt
Copy link
Member

@jneira That name looks extremely odd. Perhaps that's an error with how the UnitId is translated to a Java class name in Eta.CodeGen.Name.

@jneira
Copy link
Collaborator

jneira commented Jun 12, 2019

In gitter @justinphan3110 has shared a stacktrace trying to execute the example, replacing the web server with standard output
stacktrace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants