You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@edgent.apache.org by dlaboss <gi...@git.apache.org> on 2017/04/20 15:54:35 UTC
[GitHub] incubator-edgent pull request #306: [Edgent-409] add analytics.math3.Aggrega...
GitHub user dlaboss opened a pull request:
https://github.com/apache/incubator-edgent/pull/306
[Edgent-409] add analytics.math3.Aggregations
- analytics.math3
- add Aggregations
- add UnivariateAggregate (general math3.json.JsonUniv...)
- add UnivariateAggregator (general math3.json.JsonUniv...)
- add package-info
- analytics.math3.stats - add tuple-type-independent stats
- add Statistics2 (general Statistics)
- add Regression2 (general Regression)
- add StorelessStatistic (general JsonStorelessStatistic)
- add package scoped Count
- add package scoped OLS (general JsonOLS)
- add Statistics2Test
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dlaboss/incubator-edgent edgent409-aggregations
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-edgent/pull/306.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #306
----
commit 99e9f1519ddf900c996449d74c3867db099dbbce
Author: Dale LaBossiere <dl...@us.ibm.com>
Date: 2017-04-20T15:35:41Z
[Edgent-409] add analytics.math3.Aggregations
- analytics.math3
- add Aggregations
- add UnivariateAggregate (general math3.json.JsonUniv...)
- add UnivariateAggregator (general math3.json.JsonUniv...)
- add package-info
- analytics.math3.stats - add tuple-type-independent stats
- add Statistics2 (general Statistics)
- add Regression2 (general Regression)
- add StorelessStatistic (general JsonStorelessStatistic)
- add package scoped Count
- add package scoped OLS (general JsonOLS)
- add Statistics2Test
commit d4374bfa98c0b489822830f2c5993d902c2eca74
Author: Dale LaBossiere <dl...@us.ibm.com>
Date: 2017-04-20T15:51:51Z
fix javadoc warnings
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-edgent pull request #306: [Edgent-409] add analytics.math3.Aggrega...
Posted by dlaboss <gi...@git.apache.org>.
Github user dlaboss commented on a diff in the pull request:
https://github.com/apache/incubator-edgent/pull/306#discussion_r112492352
--- Diff: analytics/math3/src/main/java/org/apache/edgent/analytics/math3/UnivariateAggregator.java ---
@@ -0,0 +1,47 @@
+/*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements. See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership. The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied. See the License for the
+specific language governing permissions and limitations
+under the License.
+*/
+package org.apache.edgent.analytics.math3;
+
+/**
+ * Univariate aggregator for tuples.
+ * This is the runtime implementation interface
+ * of {@link UnivariateAggregate} defined aggregate.
+ */
+public interface UnivariateAggregator {
+
+ UnivariateAggregate getAggregate();
+
+ /**
+ * Clear the aggregator to prepare for a new aggregation.
+ * @param n Number of tuples to be aggregated.
+ */
+ void clear(int n);
+
+ /**
+ * Add a value to the aggregation.
+ * @param value Value to be added.
+ */
+ void increment(double value);
+
+ /**
+ * Get the aggregation result. It may be a NaN (empty collection, etc) or Infinite value.
+ * @return the result
+ */
+ double getResult();
--- End diff --
fyi, getResult() is different style than the "add to result" JsonUnivariateAggregator.result(..., JsonObject).
Having getResult(), rather than "add to result" result(ResultMap), enables subsequent
of UnivariateAggregate/UnivariateAggregator by JsonAnalytics without requiring
its implementation to use ResultMap.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-edgent pull request #306: [Edgent-409] add analytics.math3.Aggrega...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-edgent/pull/306
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---