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

Remove a note about cloudpickle support being experimental #34199

Closed
wants to merge 2,952 commits into from

Conversation

tvalentyn
Copy link
Contributor

No description provided.

rohitsinha54 and others added 30 commits January 31, 2025 12:43
… aggregate a collection of string sequences (FQNs) with a limited size.
* Add python ml extras

* Create clear bounds on transformers suite and add latest test

* Add docs
Introduce a BoundedTrie metric which is used to efficiently store and aggregate a collection of string sequences (FQNs) with a limited size.

It is recommended to review this PR by commits.

BoundedTrie is a space-saving way to store many string sequences (like FQN/file paths). It acts like a tree with branches, holding sequences within a size limit. It can efficiently add, combine, and search and perform trimming of children when the size increases beyond defined max.

Let's say we want to store these sequences, with a size limit of 3:

"folder1/file1.txt"
"folder1/file2.txt"
"folder2/file3.txt"
Here's how the BoundedTrie might look:

root
  - folder1
     - file1.txt
     - file2.txt
  - folder2
     - file3.txt 
If we try to add "folder1/file4.txt", the trie might trim to "folder1", dropping all children to stay within the size limit.

This will be used to replace the StringSet metric for lineage tracking for very large lineage graphs to overcome the size limits.
* The Bag Partition is now configurable.

Configuring the number of partitions in the Dask runner is very important to tune performance. This CL gives users control over this parameter.

* Apply formatter.

* Passing lint via the `run_pylint.sh` script.

* Implementing review feedback.

* Attempting to pass lint/fmt check.

* Fixing isort issues by reading CI output.

* More indentation.

* rm blank like for isort.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment