-
Notifications
You must be signed in to change notification settings - Fork 61
Conversation
@@ -20,6 +20,9 @@ typedef dynamic FieldAccessor(dynamic item, dynamic key); | |||
/** | |||
* Given list of items, dimensions and facts, compute | |||
* aggregates (COUNT, SUM, MIN, MAX) for facts at each dimension level. | |||
* | |||
* [Design/API Document] | |||
* (https://docs.google.com/a/google.com/document/d/1m6xGU9KF3nWPRjwILGRFhoi1_9mCCkCPIs0OxsaoQMU/view) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this comment - cannot access the document, possibly add a TODO to document the design and API.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
Implement charts api v2
_registerListeners() { | ||
_dataSubscriptions.dispose(); | ||
|
||
if(_data is ChartDataObservable) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need merge, because ChartDataObserverable does not exist anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
Please merge and update the pull request. |
merge charts.dart Modification base on comments
…/charted into filter-transformer Conflicts: lib/charts/charts.dart lib/charts/transformers/aggregation_transformer.dart lib/charts/transformers/filter_transformer.dart test/charts/transformers/aggregation_transformer_test.dart test/charts/transformers/chain_transform_test.dart
Adding filter transformer and tests.
No description provided.