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

Cleanup performance output add a focused benchmark test for comparisons #539

Closed
wants to merge 970 commits into from

Conversation

jamie256
Copy link
Contributor

Description

The cProfile stats needed a StringIO to output with the logger, and added a way to iterate over different size dataframes.

This PR needs some follow up to get the right number of columns or other data types (other than floats), but let's iterate on this.

General Checklist

  • Tests added for this feature/bug
    if it was a bug, test must cover it.
  • Conform by the style guides, by using formatter
  • Documentation updated
  • (optional) Please add a label to your PR

lalmei and others added 30 commits July 29, 2021 09:51
testing unicode

unicode tracking

add position test
add None default value

set default values to None instead of bool
[ADD] Implement table data with hbs

[FIX] Remove unused code

[ADD] Two CDN and all-in-one versions of hbs templates

[FIX] Typo in condition
compile and render page

add register helpers

check profile length
FelipeAdachi and others added 27 commits April 14, 2022 09:13
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Jamie Broomall <88007022+jamie256@users.noreply.github.com>
Removing the old version to clone the java repo
…ons, notebooks, tests.

removing/merging/renaming existent notebooks: segmentation,constraints,profilevisualizer
* Update version

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Melly Burns <9437821+TheMellyBee@users.noreply.github.com>
NotebookProfileViewer rename to NotebookProfileVisualizer
* This should reflect the correct errors. Exposing the full error message may be too much though.
* Found the last two places that error message pops up and exchanged for the actual error message.

Please note: we might want to design this to check different types of errors first. This is a step in the right direction
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Jamie Broomall <jamie@whylabs.ai>
* Fixes #531 by detecting if in WSL and prefixing the output path accordingly before opening with webbrowser
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet