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

End-to-end dfencoder test #777

Merged
merged 103 commits into from
Mar 24, 2023
Merged

End-to-end dfencoder test #777

merged 103 commits into from
Mar 24, 2023

Conversation

dagardner-nv
Copy link
Contributor

Description

  • Adds @hsin-c's end-to-end test script to our unittests along with associated data
  • New data file signed-off by @BartleyR

Requires #763 to be merged first

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@dagardner-nv
Copy files as-is from https://github.com/nv-morpheus/dfencoder @ bba45b6
53da129
@dagardner-nv
Insert Apache CR header, along with the original license
b5dd565
@dagardner-nv
Original license file for dfencoder
fd22899
@dagardner-nv
Remove dfencoder dep
a76367d
@dagardner-nv
Find/replace dfencoder imports
2956212
@dagardner-nv
Update CI containers to ensure dfencoded isn't installed and prevent …
2a5d7bd 
…tests from accidentally passing due to presence of old package
@dagardner-nv
First pass at tests for dfencoder
08053d0
@dagardner-nv
wip
dbeb1d8
@dagardner-nv
wip
57423ed
@dagardner-nv
more tests
aeca28f
@dagardner-nv
Tests for NullScaler
927bfff
@dagardner-nv
Consolidate duplicated random seeding code into a single utility func…
4315d16 
…tion
@dagardner-nv
Tests for dfencoder's EncoderDaraframe
75adf73
@dagardner-nv
wip
7b8f1c9
@dagardner-nv
Fix missing attribute error for default value of interpret_activation…
d386f6c 
… method
@dagardner-nv
Remove unused imports
f39702c
@dagardner-nv
manual_seed fixture needs to be per-function so that tests are not im…
4e71cdc 
…pacted by execution order
@dagardner-nv
Explicit fixture usage
8c441ae
@dagardner-nv
tests for CompleteLayer class
29d3fd4
@dagardner-nv
wip
a0c924e
@dagardner-nv
Merge branch 'branch-23.03' into david-migrate-dfencoder
28701b9
@dagardner-nv
Fix bad copy/paste
ed48a03
@dagardner-nv
wip
7664657
@dagardner-nv
Test for get_scaler
46c5501
@dagardner-nv
Make a fixture for reading the most commonly used test file into a da…
7b15ce0 
…taframe
@dagardner-nv
Add test for init_numeric
67a5417
@dagardner-nv
Add comment about poorly named attribute
95f0dcb
@dagardner-nv
wip
ca4bc74
@dagardner-nv
Document num_names attribute
95622d6
@dagardner-nv
wip
1f3c8c1
dagardner-nv and others added 23 commits March 20, 2023 16:44
@dagardner-nv
Merge branch 'david-migrate-dfencoder' into david-hsin-dfencoder-tests
17684b3
@dagardner-nv
Merge branch 'branch-23.03' into david-migrate-dfencoder
2a36082
@dagardner-nv
Work-around for rapidsai/cudf#12862
800ac56
@dagardner-nv
Use CUDA_PATH and also set CUDAHOSTCXX
527cc3d
@dagardner-nv
wip [no ci]
3d822c7
@dagardner-nv
wip
fcf3d7a
@dagardner-nv
Merge branch 'branch-23.03' into david-migrate-dfencoder
b70e338
@dagardner-nv
Add build-essential to driver image
55d72c9
@dagardner-nv
Merge branch 'branch-23.03' into david-migrate-dfencoder
b60ed54
@dagardner-nv
Add dill to deps
bfeff13
@dagardner-nv
Merge branch 'david-migrate-dfencoder' into david-hsin-dfencoder-tests
43c437a
@mdemoret-nv
Improving the df_type fixture to pull from marks
0844af1
@mdemoret-nv
Adding marks to entire file for only python and pandas
abf0ad8
@dagardner-nv
Sort imports
2c0f2e5
@dagardner-nv
Merge branch 'branch-23.03' into david-migrate-dfencoder [no ci]
3c047c5
@dagardner-nv
Add comment with issue on removing dfencoder
aef972a
@dagardner-nv
Rename df fixture to filter_probs_df
3d79e76
@dagardner-nv
Rename dfencoder_tests to dfencoder [no ci]
01c95ac
@dagardner-nv
Merge branch 'branch-23.03' into david-migrate-dfencoder [no ci]
2b343e1
@dagardner-nv
Replace overloaded df_type fixture with marks [no ci]
729680f
@dagardner-nv
df fixture now uses the filter_probs_df ficture removing some duplica…
945d336 
…ted code, revert change to yields
@dagardner-nv
Fix bad copy/paste
6c4033a
@dagardner-nv
Merge branch 'david-migrate-dfencoder' into david-hsin-dfencoder-tests
4422f44
@dagardner-nv dagardner-nv added 3 - Ready for Review and removed Merge After Dependencies PR is completed and reviewed but depends on another PR; do not merge out of order labels Mar 24, 2023
@mdemoret-nv
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit 9e4065f into nv-morpheus:branch-23.03 Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BartleyR BartleyR BartleyR approved these changes

@mdemoret-nv mdemoret-nv mdemoret-nv approved these changes

Assignees
No one assigned
Labels
improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Morpheus Boards
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dagardner-nv @mdemoret-nv @BartleyR