[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-13 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/729 All tests pass with `docker/build.sh -t -n -i` VOTE +1 ---

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-12 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/729 just pushed a fix for the double issueit was easy - pretty much as @FlorianHockmann deduced. re-running docker build now. ---

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-10 Thread FlorianHockmann
Github user FlorianHockmann commented on the issue: https://github.com/apache/tinkerpop/pull/729 > The .net build is failing: `Double` seems to be a type that we didn't have in the traversal API until now. It should be simply mapped to `double` in C#. This should only require

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-10 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/729 The .net build is failing: ```text Process/Traversal/GraphTraversal.cs(500,36): error CS0246: The type or namespace name 'Double' could not be found (are you missing a using directi

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-09 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/729 I just unzipped the jar and the META-INF/ does not contain it. Just a pom.xml in there. ---

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-09 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/729 Does exp4j contain a NOTICE file in its jar (usually in `/META-INF`)? ---

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-05 Thread twilmes
Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/729 This is going to come in super handy. A hearty VOTE: +1 ---

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-04 Thread okram
Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/729 ``` [INFO] [INFO] BUILD SUCCESS [INFO]

[GitHub] tinkerpop issue #729: TINKERPOP-1632: Create a set of default functions

2017-10-04 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/729 You can add a static compiled regex pattern: ``` private static final Pattern EQUATION_PATTERN = Pattern.compile("\\b(?!abs|acos|asin|atan|cbrt|ceil|cos|cosh|exp|floor|log|log10|log2|