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

vdk-jupyter: convert job operation #2406

Merged
merged 17 commits into from
Jul 24, 2023

Conversation

duyguHsnHsn
Copy link
Collaborator

@duyguHsnHsn duyguHsnHsn commented Jul 12, 2023

What:
This change includes:

  • implementing the actual transform operation
  • adding an archive of the old job to the parent directory of the job
  • adding a handler that captures the request from the front-end for the transform operation
  • adding the necessary UI components

More info for the operation can be found: https://github.com/vmware/versatile-data-kit/wiki/VDK-Jupyter-Integration-Convert-Job-Operation

Testing: python unit tests, TBD: jest tests

Example of how transformed job looks like:
Screenshot 2023-07-14 at 14 27 21
Screenshot 2023-07-14 at 14 27 30
Screenshot 2023-07-14 at 14 28 07

This is how a transformed job would look like, no UI changes are introduced in this PR

Signed-off-by: Duygu Hasan hduygu@vmware.com

@antoniivanov
Copy link
Collaborator

Please provide screenshot highlighting any UI changes

Copy link
Collaborator

@murphp15 murphp15 left a comment

Choose a reason for hiding this comment

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

documentation on the classes is v good

@duyguHsnHsn duyguHsnHsn changed the title vdk-jupyter: transform job operation vdk-jupyter: convert job operation Jul 19, 2023
@duyguHsnHsn duyguHsnHsn force-pushed the person/hduygu/vdk-transform-job-operation branch from 24fae57 to 0ee684d Compare July 24, 2023 07:37
@duyguHsnHsn duyguHsnHsn force-pushed the person/hduygu/vdk-transform-job-operation branch from a6b10df to cf4e4c4 Compare July 24, 2023 09:32
@duyguHsnHsn duyguHsnHsn enabled auto-merge (squash) July 24, 2023 09:37
@duyguHsnHsn duyguHsnHsn merged commit f9a290b into main Jul 24, 2023
7 checks passed
@duyguHsnHsn duyguHsnHsn deleted the person/hduygu/vdk-transform-job-operation branch July 24, 2023 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants