Niels Bantilan

Results 471 comments of Niels Bantilan

If we take inspiration for zenml, they have a simple `enable_mlflow` decorator: https://github.com/zenml-io/zenml/tree/main/examples/mlflow_tracking#-how-the-example-is-implemented

hey @fubuloubu just following up here. Thanks for making the issue! I can help out with some of this task list, but it'll probably be ~1-2 months before I can...

Hi @trishitapingolia this issue isn't directly related to this repo (the website is implemented in webflow), but since we don't have a specific way of gathering feedback for the union.ai...

hi @adityyyy, please setup and run the pre-commit hook [here](https://github.com/unionai-oss/unionml/blob/main/.pre-commit-config.yaml#L36-L44) should re-render the notebook automatically (the .ipynb files shouldn't actually be edited)

hey @adityyyy lemme know if you need help with the pre-commit commands, you can follow the instructions here: https://unionml.readthedocs.io/en/stable/contributing_guide.html#pre-commit-setup

@peridotml you can `#self-assign` this to yourself. Btw, we use a custom docs building process for the tutorials (which I still need to document) But basically if you create a...

thanks @adityyyy, please update the myst markdown file instead of the notebook: https://github.com/unionai-oss/unionml/blob/main/docs/tutorials/quickdraw.md Then the `pre-commit` hook [here](https://github.com/unionai-oss/unionml/blob/main/.pre-commit-config.yaml#L36-L44) should render the notebook automatically (the `.ipynb` files shouldn't actually be edited)

closing this PR because edits don't contribute much to clarity of the writing

Writing feedback here for posterity. ## Draft Proposal 1. Create a `TFRecordFile` type that extends `FlyteFile` to include an additional record type `FlyteFile["tfrecord"]` for serializing/deserializing tfrecords, which handles `tf.train.Example` task...

> But I don't think it's possible to streamline this into a type; so a better alternative will be to enable users to provide kwargs and let them apply additional...