Skip to content
Snippets Groups Projects

exponentially decay message frequency in live_info

Merged Jorn Hoofwijk requested to merge limit_message_queue into master
All threads resolved!

This way, you would not have to wait for all messages to be displayed after you open your laptop in the morning when you run an simulation overnight on the cluster.

notice this is a proof of concept, the randomness is not really important, but could be removed with if we keep track of the last printed message. Also it depends on status.comm.kernel.iopub_thread._events which I suppose is meant for internal usage (indicated by the underscore before _events.

Both these should be possible to mitigate when more time is spend on those parts.

Edited by Bas Nijholt

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Joseph Weston
    • Resolved by Bas Nijholt

      If I understand correctly what we want to achieve is to stop posting updates to the runner info when there is no client connected.

      It would be preferable to find a way to do this without private API (we should ask on Gitter or sth about how to apply "backpressure" in ipykernel).

      We should accept this change, as it does improve the user experience and the "ugliness" is hidden away in the implementation. Minimally I would like to see a test for the private API that we hit before I give a thumbs up (we will have to test specifically against the most recent version of ipykernel, and maybe master also, but this can be a separate MR.).

  • Bas Nijholt added 1 commit

    added 1 commit

    • 40f48876 - test private attribute used in 'live_info'

    Compare with previous version

  • Bas Nijholt removed assignee

    removed assignee

  • Bas Nijholt
  • Bas Nijholt marked as a Work In Progress

    marked as a Work In Progress

  • I've asked for some advice here.

    Edited by Bas Nijholt
  • Bas Nijholt added 53 commits

    added 53 commits

    Compare with previous version

  • Bas Nijholt added 1 commit

    added 1 commit

    • 43d2b92a - test private attribute used in 'live_info'

    Compare with previous version

  • Bas Nijholt unmarked as a Work In Progress

    unmarked as a Work In Progress

  • assigned to @jbweston

  • Bas Nijholt added 1 commit

    added 1 commit

    • 7f936b7c - test private attribute used in 'live_info'

    Compare with previous version

  • Bas Nijholt resolved all discussions

    resolved all discussions

  • merged

  • Joseph Weston mentioned in commit a6dae434

    mentioned in commit a6dae434

  • mentioned in issue #125 (closed)

  • Please register or sign in to reply
    Loading