[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15801986#comment-15801986
 ] 

ASF GitHub Bot commented on EDGENT-200:
---

Github user asfgit closed the pull request at:

https://github.com/apache/incubator-edgent/pull/258


> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: in_equals_out_tuple_count.png, 
> in_not_equal_to_out_tuple_count.png, stream_hover_current.png, 
> tuple_count_static_layer.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15801734#comment-15801734
 ] 

ASF GitHub Bot commented on EDGENT-200:
---

Github user queeniema closed the pull request at:

https://github.com/apache/incubator-edgent/pull/258


> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: in_equals_out_tuple_count.png, 
> in_not_equal_to_out_tuple_count.png, stream_hover_current.png, 
> tuple_count_static_layer.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15770985#comment-15770985
 ] 

ASF GitHub Bot commented on EDGENT-200:
---

GitHub user queeniema opened a pull request:

https://github.com/apache/incubator-edgent/pull/258

[EDGENT-200] Console: Add missing tuple counts



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/queeniema/incubator-edgent EDGENT-200

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-edgent/pull/258.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 #258


commit 8b34651a3ba489dc6adad58048fe5deb0ffa0285
Author: Queenie Ma 
Date:   2016-12-22T20:09:43Z

[EDGENT-200] Console: Add missing tuple counts




> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: in_equals_out_tuple_count.png, 
> in_not_equal_to_out_tuple_count.png, stream_hover_current.png, 
> tuple_count_static_layer.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15550199#comment-15550199
 ] 

ASF GitHub Bot commented on EDGENT-200:
---

Github user asfgit closed the pull request at:

https://github.com/apache/incubator-edgent/pull/211


> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: in_equals_out_tuple_count.png, 
> in_not_equal_to_out_tuple_count.png, stream_hover_current.png, 
> tuple_count_static_layer.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15549477#comment-15549477
 ] 

ASF GitHub Bot commented on EDGENT-200:
---

GitHub user queeniema opened a pull request:

https://github.com/apache/incubator-edgent/pull/211

[WIP] [EDGENT-200] Console: Tuple count updates

- [X] Add in/out tuple counts
- [ ] Add tuple counts for the _Static flow_ layer

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/queeniema/incubator-edgent EDGENT-200

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-edgent/pull/211.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 #211


commit dae7e4c70c2ddfeb6cb765a371cdc12c1a554a5e
Author: Queenie Ma 
Date:   2016-10-05T17:45:04Z

[EDGENT-200] Console: Add in/out tuple counts




> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: stream_hover_current.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[Dale LaBossiere (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15533681#comment-15533681
 ] 

Dale LaBossiere commented on EDGENT-200:


The simplest #OUT != #IN example is TStream.filter()
Many (most?) other TStream/TWindow methods also enable the application to yield 
#OUT!=#IN (e.g., map() can yield 0 or 1 tuple).

> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: stream_hover_current.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[Dale LaBossiere (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15533609#comment-15533609
 ] 

Dale LaBossiere commented on EDGENT-200:


Re "Regarding the previous comment, I think it would be safe to remove the 
Tuple count field column from oplet hovers...", I think there's value in 
continuing to have the overall tuple count column for an oplet.  As a user I 
don't want to have to "do the math" to compute the aggregate values when 
there's >1 Source/Target stream :-)

> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: stream_hover_current.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (EDGENT-200) Console: stream hovers in oplet kind layout lack tuple counts

[Queenie Ma (JIRA)]

[ 
https://issues.apache.org/jira/browse/EDGENT-200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15533242#comment-15533242
 ] 

Queenie Ma commented on EDGENT-200:
---

I see that stream hovers already have tuple count information. See attached: 
stream_hover_current.png. So, this issue may not be relevant anymore.

But, I like the idea of adding tuple counts to hover popups in the _Static 
flow_ layer.

Regarding the previous comment, I think it would be safe to remove the _Tuple 
count_ field column from oplet hovers since this information is now available 
in the _Source streams_ and _Target streams_ columns as part of 
[EDGENT-178|https://issues.apache.org/jira/browse/EDGENT-178]. Each 
source/target stream now lists the number of tuples flowing through it, as well 
as alias and tag information. With this, we can see the number of tuples 
flowing in/out of an oplet.



> Console: stream hovers in oplet kind layout lack tuple counts
> -
>
> Key: EDGENT-200
> URL: https://issues.apache.org/jira/browse/EDGENT-200
> Project: Edgent
>  Issue Type: Improvement
>  Components: Console
>Reporter: Dale LaBossiere
>Assignee: Queenie Ma
>Priority: Minor
>  Labels: newbie
> Attachments: stream_hover_current.png
>
>
> stream hovers lack tuple count.  oplet hovers have them.
> related, wish the stream/oplet hovers in the static layout had tuple cnt info 
> too.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)