Eduardo Apolinario

Results 268 comments of Eduardo Apolinario

If I understand your concern, you want the ability to remove specific versions of workflows and tasks, correct? We avoided a whole class of errors by deciding to not expose...

Thank you, can you fix the conficts, the lint warnings and make sure to run `make -C flyteadmin regenerate` from the root of the repo?

Let's investigate two (complementary) approaches: 1. validate podspec at registration time in flytekit 2. Use https://pkg.go.dev/k8s.io/kubernetes/pkg/apis/core/validation to validate podspecs in flyteadmin

Code Review Agent Run #3d8b84 Actionable Suggestions - 1 plugins/flytekit-deck-standard/tests/test_renderer.py - 1 Consider removing duplicate test case · Line 44-45 Additional Suggestions - 1 plugins/flytekit-deck-standard/flytekitplugins/deck/renderer.py - 1 Consider removing redundant...

Changelist by Bito This pull request implements the following key changes. Key Change Files Impacted Feature Improvement - Enhanced ydata-profiling Integration - [`dev-requirements.in`](https://github.com/flyteorg/flytekit/pull/2921/files#diff-c7b88c3f2e7c91477a6561647bdd71098343384fd676b3c3e6e7f16d87a22147) - Added lxml dependency for HTML parsing - [`renderer.py`](https://github.com/flyteorg/flytekit/pull/2921/files#diff-4838a9c9e4363a5de0ad4b26e37ac07060fd5f22139fd2439ec812810d9b68a6)...

Need to handle the case of the "dangling" node, i.e. how to force the evaluation of nodes that are not an upstream dependency of the graph.

@otarabai , can you fix the lint error? Also, if you merge master the errors in the `greatexpectations` should go away.