Thomas J. Fan
Thomas J. Fan
## Why are the changes needed? With `1.13.0`, the default image builder does not allow `flytekit` to be specified to a specific version. For example, the following will fail: ```...
## Why are the changes needed? This PR adds a comet-ml plugin to flytekit. ## What changes were proposed in this pull request? This PR adds a `comet_ml_init` task decorator...
## Tracking issue Closes https://github.com/flyteorg/flyte/issues/3681 ## Why are the changes needed? Type checkers do not work with `flytekit` right now. ## What changes were proposed in this pull request? With...
This PR adds a comet-ml example. To fully run the example, you need to use this `ImageSpec`, because the plugin is in a PR: ```python comet_ml_package = ( "git+https://github.com/thomasjpfan/flytekit.git@" "d04d0129f99eb7e7edcffeae04d4f401f49be6d6#subdirectory=plugins/flytekit-comet-ml"...
#### Context - [ ] add a new feature - [ ] fix a bug - [ ] update tests and/or documentation - [x] other (Expose new feature in recipe...
Currently scikit-learn is compiled on the GPU runner, which takes [2m 24s](https://github.com/scikit-learn/scikit-learn/actions/runs/10451046790/job/28936627899?pr=29475) from the GPU runner. This PR moves the scikit-learn builds a wheel using the CPU runner and then...
#### Reference Issues/PRs Fixes https://github.com/scikit-learn/scikit-learn/issues/29662 #### What does this implement/fix? Explain your changes. The rank of `X.T @ X` is bounded above by `min(n_samples, n_features)` and not just `n_features`. For...
#### Reference Issues/PRs Fixes https://github.com/scikit-learn/scikit-learn/issues/20804 #### What does this implement/fix? Explain your changes. This PR implements `__sklearn_tags__`, while also keeping backward compatibility. The idea is to **not** walk the MRO...
### Motivation: Why do you think this is important? Currently, creating a type transformer requires learning about the [TypeTransformer](https://github.com/flyteorg/flytekit/blob/be574ddaa01ddb8db04e4f3dde4bef71274fdae0/flytekit/core/type_engine.py#L122) API which requires someone to learn about Flyte Literals, Flyte Context,...
### Describe the bug Currently, `--copy-all`, the image builders, and register uses `.gitignore` to ignore files. A common use case for register is to include compiled binaries into the image...