Skip to content

Some microbenchmarks stopped sending data  #45732

Closed

Description

@LongCatIsLooong

2019-11-27-140830_2529x473_scrot

Started happening since 5df4b7d

@blasten @liyuqian

Activity

  1. Hixie commented on Nov 27, 2019

    @Hixie
    Contributor

    @liyuqian suggests this is because flutter run xxx.dart is ignoring the xxx.dart. If this is true then this is a stop-ship regression.

  2. added this to the milestone on Nov 27, 2019
  3. blasten commented on Nov 27, 2019

    @blasten

    @Hixie I'm looking at this issue. I doesn't seem flutter run is ignoring the entrypoint. As you noted, it's very likely that this issue is caused by #45307 (comment).

  4. xster commented on Nov 27, 2019

    @xster
    Member

    flutter run -t xxx seems to work for me on head as well`

  5. blasten commented on Nov 27, 2019

    @blasten

    The issue is that the test relies on the flutter run stdout. In verbose mode, flutter run stdout also includes the device's stdout (from the device logger) which used to skip logs that were older than the flutter run command invocation. In my change, I forwarded all the device logs that have the "flutter" tag, so that isn't what the test expects.

  6. liyuqian commented on Nov 27, 2019

    @liyuqian
    Contributor

    I just confirmed that flutter run xxx.dart worked for me too. Sorry for giving you the wrong suggestion because I tried to modify the running target in microbenchmarks.dart and saw the result of dart bin/run.dart -t microbenchmarks to be not changed. Now I understand that the result doesn't change because we're always reading the old log...

  7. added
    a: tests"flutter test", flutter_test, or one of our tests
    c: regressionIt was better in the past than it is now
    and removed on Nov 27, 2019
  8. Hixie commented on Nov 27, 2019

    @Hixie
    Contributor

    Ok, good. Removing panic markers.

    Can we return to the previous behaviour? We didn't have the old behaviour just randomly, it was very much intentional that flutter logs and flutter run not give you old logs. :-)

  9. blasten commented on Nov 28, 2019

    @blasten

    The microbenchmarks are back!

  10. Hixie commented on Nov 28, 2019

    @Hixie
    Contributor

    Yay! Thanks for fixing this on your day off! Please definitely don't feel you must do that, but it is appreciated! :-)

  11. github-actions commented on Aug 13, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  12. locked as resolved and limited conversation to collaborators on Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    a: tests"flutter test", flutter_test, or one of our testsc: regressionIt was better in the past than it is nowtoolAffects the "flutter" command-line tool. See also t: labels.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions