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.

Sign up for GitHub

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

Jump to bottom

Replicate ludwig type inference, using the whylogs dataset profile. #2743

Merged
merged 32 commits into from
Nov 11, 2022
Merged

Replicate ludwig type inference, using the whylogs dataset profile. #2743

merged 32 commits into from
Nov 11, 2022

Conversation

justinxzhao
Copy link
Collaborator

No description provided.

@justinxzhao
Minor (temporary) adjustments to the twitterbots example.
a835633
@justinxzhao
in progress, profiling
dcddb82
@justinxzhao
in progress, profiling
2dc7ae5
@justinxzhao
Merge with master
4bdd10a
@justinxzhao
Remove irrelevant notebooks
4df7566
@justinxzhao
Refactored protos, added protolint, precommits, and a unit test with …
3464f0a 
…adult census income.
@justinxzhao
Diff cleanup
a3b1d3c
@justinxzhao
Remove DatasetInfo legacy code.
378436f
@justinxzhao
Remove pytest profiling.
c521c2e
@justinxzhao
Add minor comment.
e8a0370
@justinxzhao
Remove custom Ludwig schemas, implement type inference on the out of …
bbfcc4c 
…the box whylogs profile, add a test for type inference.
@justinxzhao
Fix bug with torchinfo summary.
2c5c60e
@justinxzhao
(mostly obsolete) notebook updates.
90f62aa
@justinxzhao
Merge with master, need to regenerate proto files
c1e7ebc
@justinxzhao
Regenerated proto files.
6e1fcf6
@justinxzhao
Split dataset_profile into api and utils files.
ddbf97d
@justinxzhao
Rename files
4677cf3
@justinxzhao
Add support for dask
1661520
@justinxzhao
Delete python notebooks
809ed10
@justinxzhao
Merge branch 'master' of github.com:ludwig-ai/ludwig into whylogs_pro…
3247a9f 
…totype
@justinxzhao
Merge branch 'master' of github.com:ludwig-ai/ludwig into whylogs_pro…
5d98e36 
…totype
@justinxzhao
Refactoring, adding synthetic data tests.
296ada9
@justinxzhao
Compute size in bytes before cap
bd6cbc5
@justinxzhao
Fix docstring.
898417f
@justinxzhao
Refactor generate_data to use generate_data_as_dataframe
718a8bc
@justinxzhao
Remove print statements
47b0841
@justinxzhao
Simplify test
6cefef0
@justinxzhao
Merge branch 'master' of github.com:ludwig-ai/ludwig into whylogs_pro…
ab76c72 
…totype
@justinxzhao justinxzhao changed the title Replicate type inference, using the whylogs dataset profile. Replicate ludwig type inference, using the whylogs dataset profile. Nov 9, 2022
@github-actions
Copy link

github-actions bot commented Nov 9, 2022 •
edited
Loading

Unit Test Results

         6 files  ±  0           6 suites  ±0   3h 34m 53s ⏱️ + 6m 2s
  3 509 tests +  4    3 428 ✔️ +  4    81 💤 ±0  0 ❌ ±0 
10 527 runs  +12  10 266 ✔️ +12  261 💤 ±0  0 ❌ ±0 

Results for commit fd2e42e. ± Comparison against base commit f885b91.

♻️ This comment has been updated with latest results.

dantreiman
dantreiman approved these changes Nov 10, 2022
View reviewed changes
Copy link
Collaborator

@dantreiman dantreiman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@justinxzhao justinxzhao merged commit 24e394f into master Nov 11, 2022
@justinxzhao justinxzhao deleted the whylogs_prototype branch November 11, 2022 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dantreiman dantreiman dantreiman approved these changes

@tgaddair tgaddair Awaiting requested review from tgaddair tgaddair is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@justinxzhao @dantreiman