Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unexpectedness and log data #450

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

antter
Copy link
Contributor

@antter antter commented Nov 24, 2021

Related Issues and Dependencies

#449

This introduces a breaking change

  • Yes
  • No

This Pull Request implements

… Explain your changes.

Description

I added a notebook to be able to view the testgrid data and the logs in a parallelized manner, so further analysis can be done. I want to examine the error messages and see if anything specifically sticks out. I also like the methods in the log_term_freq notebook in the gsweb data source and will likely apply some to this.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@sesheta sesheta added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 24, 2021
@sesheta
Copy link
Contributor

sesheta commented Nov 24, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign michaelclifford after the PR has been reviewed.
You can assign the PR to them by writing /assign @michaelclifford in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Nov 24, 2021
@oindrillac oindrillac self-requested a review December 2, 2021 15:45
@antter antter changed the title [WIP] Unexpectedness and log data Unexpectedness and log data Dec 17, 2021
@sesheta sesheta removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants