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

fix: dimensionality per data type blob and be disabled in UI #371

Merged
merged 1 commit into from
Apr 12, 2022

Conversation

eugenejahn
Copy link
Contributor

@eugenejahn eugenejahn commented Apr 12, 2022

Signed-off-by: Eugene Jahn eugenejahnjahn@gmail.com

fix dimensionality per data type blob and be disabled in UI. For the blob input, we will set the dimensionality based on the blob type(if dimensionality from blob type is empty, we set to BlobDimensionality.SINGLE)

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

add the dimensionality from typeDefinition?.literalType?.blob?.dimensionality
disable the select component

Tracking Issue

fixes https://github.com/flyteorg/flyteconsole/issues/336

Signed-off-by: Eugene Jahn <eugenejahnjahn@gmail.com>
@codecov
Copy link

codecov bot commented Apr 12, 2022

Codecov Report

Merging #371 (14b12c1) into master (1ed28c4) will increase coverage by 0.00%.
The diff coverage is 83.33%.

@@           Coverage Diff           @@
##           master     #371   +/-   ##
=======================================
  Coverage   66.43%   66.43%           
=======================================
  Files         405      405           
  Lines        9056     9060    +4     
  Branches     1597     1600    +3     
=======================================
+ Hits         6016     6019    +3     
- Misses       3040     3041    +1     
Impacted Files Coverage Δ
src/components/Launch/LaunchForm/BlobInput.tsx 77.77% <83.33%> (-0.49%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a277e0...14b12c1. Read the comment docs.

Copy link
Contributor

@anrusina anrusina left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you for doing that :)

@flyte-bot
Copy link
Collaborator

🎉 This PR is included in version 0.51.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

[BUG] Dimensionality - should be set per data type blob and be disabled in UI.
3 participants