-
Notifications
You must be signed in to change notification settings - Fork 56
Adding UniSRec model implemented on lightweight class hierarchy with pytorch preprocessing #306
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
Merged
Merged
Changes from 18 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
d4c9df0
feat: add fast_transformers module with FlatSASRec and UniSRec models
TOPAPEC 6c875b3
feat: make UniSRec fully configurable
TOPAPEC 3cec1e0
Fast gpu preprocessing and good metrics
aa015f8
Tests + comparison
e24fec3
add changelog, fixed gpu model load
7d3850b
Formatting
f2fdfe5
feat: add ONNX export, hash ID mapping, and map_item_ids
TOPAPEC 6809160
Simplify UniSRec: remove 3-phase training, hash IDs, ranking.py
TOPAPEC 45ed8ae
Clean up UniSRec: remove dead code, add GPU metrics
TOPAPEC 757a909
fix: address review blockers and majors for UniSRecModel
TOPAPEC 17a90c5
fix: lint, val+non-softmax loss, device-aware negatives, align_embedd…
TOPAPEC 387c2d0
fix: remove unnecessary dtype preservation in align_embeddings
TOPAPEC fa91d5a
feat: use GPU for build_sequences in fit() when CUDA available
TOPAPEC 7fdef50
fix: explicit device param instead of auto-CUDA in fit()
TOPAPEC b43f4c5
fix: pass all 7 repo linters (mypy, isort, black, flake8, codespell, …
TOPAPEC 13e71a8
fix: pass all 7 repo linters for fast_transformers module
TOPAPEC 65ef231
docs: add UniSRec tutorial notebook with ML-20M dataset
TOPAPEC 204a825
Delete rectools/fast_transformers/unisrec/demo_kion.md
TOPAPEC 248bae0
Address PR #306 review: rename nets, fix security/checkpoint/ONNX/pre…
16df93f
Remove stale xfail on test_different_sequence_lengths (dynamic_axes w…
56e3a27
Fix black formatting in test_unisrec_model.py
39b89d4
Fix N806 lint ignore: move from global tests/* to local test_metrics.…
3aaef75
Add type: ignore[import-untyped] for requests in benchmark script
54bb866
Fix remaining review items: make_ffn docstring, N812 violation, paddi…
8ffd648
Remove stale type: ignore comments flagged by mypy
63898f7
fix: mypy type errors for requests import and Trainer callbacks
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -95,4 +95,8 @@ benchmark_results/ | |
| *.dat | ||
|
|
||
| # CatBoost | ||
| catboost_info/ | ||
| catboost_info/ | ||
|
|
||
| # Dev artifacts | ||
| *.pt | ||
| data/* | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.