dfdx/Espresso.jl

Compatibility with Julia 1.0

Closed this issue · 7 comments

I get this error in Julia 1.0 but not in Julia 0.7. It would be nice to have full compatibility with Julia 1.0.

dfdx commented

Can you please test it with ]add Espresso#master? I might have pushed a few commits after the latest release.

Yes, master branch allows tests to pass in Julia 1.0. Thanks! I'll wait for the next release to update the REQUIRE file. Please ping me when you do it.

dfdx commented

Here's a PR to METADATA. Once it's merged, a new version should be available.

Thanks. I'll be following it. Why isn't it fully compatible? Is it related to the dependency on MacroTools?

dfdx commented

I think you misread release notes as "not fully compatible" though actually it says "now fully compatible". Or at least existing tests don't uncover any incompatibility :)

As for MacroTools, I've removed it from dependencies for now and just copied what I needed to sugar.jl file.

dfdx commented

By the way, PR is merged, so I'm closing this issue. Please let me know if you find any other issues.

Indeed, I misread it; thanks any way!