tensorflow-haskell

eeva/tensorflow-haskell

Fork 0

mirror of https://github.com/tensorflow/haskell.git synced 2024-12-26 11:39:46 +01:00

Commit graph

Author	SHA1	Message	Date
Judah Jacobson	1539783ee5	Update type constraints to work around a ghc-8 bug. (#47 ) Also removes all the ghc-8-specific logic in the .cabal files. ghc-8 has issues with deeply nested tuples of constraints. We can work around it by: - Changing TensorTypes to a regular class. This required FlexibleContexts. (But we'll probably need it anyway when we support heterogeneous tensor lists.) - Specializing NoneOf for long type lists. For more details, see: https://ghc.haskell.org/trac/ghc/ticket/12175. Also added 'directory' to tensorflow-core-ops' dependencies since it's used in the Setup script. One more step towards fixing #38.	2016-11-28 21:15:09 -08:00
Judah Jacobson	5b4017e31b	Fix the build on ghc-8.0.1 (#38 ). (#40 ) Two issues: - The definition of `\\` was missing parentheses. It was probably a bug that this used to worked in ghc-7.10. - Set `-fconstraint-solver-iterations=0` to work around https://ghc.haskell.org/trac/ghc/ticket/12175. It looks like we can trigger that bug when defining a significantly complicated op. Specifically, our type shenanigans ("OneOf") along with lens setters (for OpDef) seem to confuse GHC. Still TODO: automate testing of different ghc versions to prevent a regression.	2016-11-21 22:20:08 -08:00
Greg Steuck	67690d1499	Initial commit	2016-10-24 19:26:42 +00:00

Author

SHA1

Message

Date

Judah Jacobson

1539783ee5

Update type constraints to work around a ghc-8 bug. (#47 )

Also removes all the ghc-8-specific logic in the .cabal files.

ghc-8 has issues with deeply nested tuples of constraints.  We can
work around it by:
- Changing TensorTypes to a regular class.  This required FlexibleContexts.
  (But we'll probably need it anyway when we support heterogeneous tensor
  lists.)
- Specializing NoneOf for long type lists.

For more details, see: https://ghc.haskell.org/trac/ghc/ticket/12175.

Also added 'directory' to tensorflow-core-ops' dependencies since it's used
in the Setup script.

One more step towards fixing #38.

2016-11-28 21:15:09 -08:00

Judah Jacobson

5b4017e31b

Fix the build on ghc-8.0.1 (#38 ). (#40 )

Two issues:
- The definition of `\\` was missing parentheses.  It was probably a bug
  that this used to worked in ghc-7.10.
- Set `-fconstraint-solver-iterations=0` to work around
  https://ghc.haskell.org/trac/ghc/ticket/12175.  It looks like we can
  trigger that bug when defining a significantly complicated op.  Specifically,
  our type shenanigans ("OneOf") along with lens setters (for OpDef) seem
  to confuse GHC.

Still TODO: automate testing of different ghc versions to prevent a regression.

2016-11-21 22:20:08 -08:00

Greg Steuck

67690d1499

Initial commit

2016-10-24 19:26:42 +00:00

3 commits