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

Enable Sem-dedup #130

Merged
merged 101 commits into from
Jul 5, 2024
Merged

Enable Sem-dedup #130

merged 101 commits into from
Jul 5, 2024

Conversation

VibhuJawa
Copy link
Collaborator

@VibhuJawa VibhuJawa commented Jun 27, 2024

Description

This PR builds on top #118 and adds the following features on top of it:

  1. End to End Bash script
  2. Improved Readme
  3. Efficient and cleaned up compute_embeddings.py, clustering.py
  4. Dask Accelerated sort_clusters.py and semdedup.py

Checklist

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

aschilling-nv and others added 21 commits June 27, 2024 03:18
* Rename CPUvsGPU.rst to cpuvsgpu.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DataCuration.rsts to datacuration.rsts

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DistributedDataClassification.rst to distributeddataclassification.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DocumentDataset.rst to documentdataset.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename Download.rst to download.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename GpuDeduplication.rst to gpudeduplication.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename KubernetesCurator.rst to kubernetescurator.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename LanguageIdentificationUnicodeFormatting.rst to languageidentificationunicodeformatting.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename PersonalIdentifiableInformationIdentificationAndRemoval.rst to personalidentifiableinformationidentificationandremoval.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename QualityFiltering.rst to qualityfiltering.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename TaskDecontamination.rst to taskdecontamination.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Update index.rst

Setting all RST files to lowercase names.

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Ignore docs for EOF fixer hook

Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>

---------

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>
Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>
Co-authored-by: Ayush Dattagupta <ayushdg95@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Added links to tutorials

Signed-off-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
@VibhuJawa VibhuJawa force-pushed the vjawa/dev_semdedup branch 2 times, most recently from 0524727 to 3179e24 Compare June 27, 2024 10:27
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Copy link
Collaborator

@ryantwolf ryantwolf left a comment

Choose a reason for hiding this comment

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

Thanks a ton @VibhuJawa, just a couple of nits and things I think you missed the first time around.

@faywang123
Copy link

faywang123 commented Jul 4, 2024

I have tested the most recent PR (using 10 data files with 12 clusters). The result is consistent with our original result. Thanks, Vibhu! This is the command:

python semdedup_example.py --input-data-dir /ads_ds3/data/SemDeDup_BenchMark/datasets/c4/realnewslike/modified --config-file configs_cf.yml

The content of configs_cf.yml:

cache_dir: "/ads_ds3/data/SemDeDup_BenchMark"
num_files: 10
id_col_name: 'id' 
id_col_type: 'int' 
input_column: 'text'
input_file_type: 'json'
embeddings_save_loc: "/ads_ds3/data/SemDeDup_BenchMark/embeddings_fbopt_c4_10_pr130"
embedding_model_name_or_path: 'facebook/opt-125m' 
embedding_batch_size: 32
embedding_max_mem_gb: 10

clustering_save_loc: "/ads_ds3/data/SemDeDup_BenchMark/results_fbopt_c4_10_pr130"
n_clusters: 12 # -- number of clusters
seed: 1234
max_iter: 100
# Kmeans can only be done with L2 using cuML.
Kmeans_with_cos_dist: False # Only False allowed

# -- which example to keep from each group of duplicates
which_to_keep: "hard"
# largest cluster size the memory is large enough to process. If the
# cluster size is larger than it, we will devide the cluster into small
# clusters and process each one separately.
largest_cluster_size_to_process: 100000
sim_metric: "cosine" # Only cosine is allowed.
eps_thresholds:  0.001 0.01
eps_to_extract: 0.001

@VibhuJawa
Copy link
Collaborator Author

Thanks so much for this @faywang123 . Appreciate all the help.

@ayushdg , Can we use @faywang123 test above and put in our testing.

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Copy link
Collaborator

@ryantwolf ryantwolf left a comment

Choose a reason for hiding this comment

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

Two nits then we're set.

config/sem_dedup_config.yaml Outdated Show resolved Hide resolved
config/sem_dedup_config.yaml Outdated Show resolved Hide resolved
@faywang123
Copy link

After a final review and test, the PR looks good to me. Thanks, @VibhuJawa for all the hard work!

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
@VibhuJawa
Copy link
Collaborator Author

@ryantwolf , Addressed the nits, let me know

Copy link
Collaborator

@ryantwolf ryantwolf left a comment

Choose a reason for hiding this comment

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

Incredible work, so excited to have this be a part of NeMo Curator

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Copy link
Collaborator

@ayushdg ayushdg left a comment

Choose a reason for hiding this comment

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

Not blocking but a couple of other suggestions:

  1. Adding an optional SemDedup import to the top level modules/__init__.py file for gpu only environments. Allowing users to do something like from nemo_curator import SemDedup
  2. Adding semantic deduplication in the list of features both in the README.md as well as a page in docs/user-guide

examples/semdedup_example.py Show resolved Hide resolved
nemo_curator/modules/semantic_dedup.py Show resolved Hide resolved
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
@VibhuJawa
Copy link
Collaborator Author

Not blocking but a couple of other suggestions:

  1. Adding an optional SemDedup import to the top level modules/__init__.py file for gpu only environments. Allowing users to do something like from nemo_curator import SemDedup

Done .

  1. Adding semantic deduplication in the list of features both in the README.md as well as a page in docs/user-guide

Added readme. docs/user-guide will be a followup.

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
nemo_curator/modules/__init__.py Outdated Show resolved Hide resolved
nemo_curator/modules/__init__.py Outdated Show resolved Hide resolved
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
@VibhuJawa VibhuJawa merged commit e557ee3 into NVIDIA:main Jul 5, 2024
3 checks passed
sarahyurick pushed a commit to sarahyurick/NeMo-Curator that referenced this pull request Jul 23, 2024
* Applying SEO Best Pratices (NVIDIA#104)

* Rename CPUvsGPU.rst to cpuvsgpu.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DataCuration.rsts to datacuration.rsts

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DistributedDataClassification.rst to distributeddataclassification.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DocumentDataset.rst to documentdataset.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename Download.rst to download.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename GpuDeduplication.rst to gpudeduplication.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename KubernetesCurator.rst to kubernetescurator.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename LanguageIdentificationUnicodeFormatting.rst to languageidentificationunicodeformatting.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename PersonalIdentifiableInformationIdentificationAndRemoval.rst to personalidentifiableinformationidentificationandremoval.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename QualityFiltering.rst to qualityfiltering.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename TaskDecontamination.rst to taskdecontamination.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Update index.rst

Setting all RST files to lowercase names.

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Ignore docs for EOF fixer hook

Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>

---------

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>
Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>
Co-authored-by: Ayush Dattagupta <ayushdg95@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Shuffle CC result on group before writing out (NVIDIA#110)

Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst (NVIDIA#113)

Added links to tutorials

Signed-off-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* first commit

Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* mv under modules dir

Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* first commit

Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* mv under modules dir

Signed-off-by: avinashvem <avem@nvidia.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* first commit

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* mv under modules dir

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* embed by cluster saved

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* id map script

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* test commit

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* add id map script

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Cleanup compute_embeddings_crossfit.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Cleanup compute_embeddings_crossfit.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Pre-commit style fixes

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* clustering_dask_crossfit.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Minor clean up to sort_clusters_crossfit.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* cleanup semdedup_crossfit

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Remove undo changes

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Remove rename changes

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix rename

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Readme formatting

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* add dask to semdedup_crossfit.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* README.md updates

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* README.md updates

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* README.md updates

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* README.md updates

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* README.md updates

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* configure max memory using a cli

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Dumb id results to parquet

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Embedding fixes

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* README.md updates

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Working end to end

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Minor  yaml fixes

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Undo changes to index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update .pre-commit-config.yaml 

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Undo changes to docs/personalidentifiableinformationidentificationandremoval.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update fuzzy_dedup.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Undo changes to docs/personalidentifiableinformationidentificationandremoval.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add end to end script in readme.md

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add type hints

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Use dask for sort_clusters

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Make sort_clusters work on MNMG scales

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Cleaned up dask shutdown

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Decrease noise in E2E scripts

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Clean up scripts

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix scripts/end_to_end_script.sh

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Some more cleanup

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add copyright

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix README.md

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Address reviews

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Make work with a SemDedupConfig

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Make work with SemDedupConfig

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Move to nemo-curator's logger

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Semdedup-extract_dedup_data.py

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Applying SEO Best Pratices (NVIDIA#104)

* Rename CPUvsGPU.rst to cpuvsgpu.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DataCuration.rsts to datacuration.rsts

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DistributedDataClassification.rst to distributeddataclassification.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename DocumentDataset.rst to documentdataset.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename Download.rst to download.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename GpuDeduplication.rst to gpudeduplication.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename KubernetesCurator.rst to kubernetescurator.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename LanguageIdentificationUnicodeFormatting.rst to languageidentificationunicodeformatting.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename PersonalIdentifiableInformationIdentificationAndRemoval.rst to personalidentifiableinformationidentificationandremoval.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename QualityFiltering.rst to qualityfiltering.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Rename TaskDecontamination.rst to taskdecontamination.rst

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Update index.rst

Setting all RST files to lowercase names.

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>

* Ignore docs for EOF fixer hook

Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>

---------

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>
Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>
Co-authored-by: Ayush Dattagupta <ayushdg95@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix bad merge

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Update index.rst

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add Module for embedding+clustering

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add sorting to clustering

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Refactor Semdup modules

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Refactor Semdup modules

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Refactor Semdup modules

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix Readme.md

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add a environment variable to silence HF warnings

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* dask-cudf fix

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* dask-cudf fix

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* dask-cudf fix

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Make config a flat file based on reviews

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add docstrings

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix argparse and seed function

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Use argparse to read config

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Move around config files

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Move around config files

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Move around config files

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Remove end_to_end_script.sh

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Append Readme

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Address Reviews

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Change config

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Make embedding creation optionally lazy

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* fix docstring

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Address Reviews and docstrings

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Address Reviews and make eps_thresholds a list of values

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Minor import fix

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Empty Commit

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Add modules to __init__ and README.md

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Fix init

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Move comment

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Empty commit to restart CI (which failed due to a download issue)

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

* Empty commit to restart CI (which failed due to a download issue)

Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>

---------

Signed-off-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>
Signed-off-by: Ayush Dattagupta <ayushdg95@gmail.com>
Signed-off-by: Vibhu Jawa <vibhujawa@gmail.com>
Signed-off-by: jgerh <163925524+jgerh@users.noreply.github.com>
Signed-off-by: avinashvem <avem@nvidia.com>
Co-authored-by: Andrew Schilling <85314306+aschilling-nv@users.noreply.github.com>
Co-authored-by: Ayush Dattagupta <ayushdg95@gmail.com>
Co-authored-by: jgerh <163925524+jgerh@users.noreply.github.com>
Co-authored-by: avinashvem <avem@nvidia.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.

7 participants