kiersten-stokes

kiersten-stokes

Member Since 4 years ago

Open Source Developer @ IBM,

Experience Points
4
follower
Lessons Completed
0
follow
Lessons Completed
2
stars
Best Reply Awards
11
repos

263 contributions in the last year

Pinned
⚡ Elyra extends JupyterLab Notebooks with an AI centric approach.
⚡ A repository of examples/sample notebooks and scenarios for Elyra
⚡ A simple neural network implementation to classify the MNIST handwritten digit set
Activity
Oct
15
1 day ago
Activity icon
issue

kiersten-stokes issue comment elyra-ai/elyra

kiersten-stokes
kiersten-stokes

Replace missing `metadata_class_name` in Component Registry schema

It was recently pointed out to me that the palette was not updating after changing a component registry as it should be after #2157. It looks like the metadata_class_name field was removed from the component registry schema in a subsequent PR.

What changes were proposed in this pull request?

Re-adds the metadata_class_name field to the component-registry schema

How was this pull request tested?

Unit tests are passing. Also tested manually by making changes to and deleting a registry and refreshing the JL page to confirm that the backend component registry cache is updating immediately after a change is made.

Developer's Certificate of Origin 1.1

   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.
kiersten-stokes
kiersten-stokes

Sorry. It looks like this was o inadvertently removed in #2109. This updated appropriately restores the entry.

I completely missed it during my review too!

Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch dictionary-fix

deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch component-registry

deleted time in 6 hours ago
Activity icon
delete
deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch finalize-components

deleted time in 6 hours ago
Activity icon
created branch

kiersten-stokes in kiersten-stokes/elyra create branch validate-image

createdAt 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch validate-image

deleted time in 6 hours ago
Activity icon
delete
deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch test-update-docs

deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch update-docs-again

deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch byo-mlx-cr-poc

deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch registry-migration

deleted time in 6 hours ago
Activity icon
delete
deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch dag-improvements

deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch list-dict-types

deleted time in 6 hours ago
Activity icon
delete
deleted time in 6 hours ago
Activity icon
delete

kiersten-stokes in kiersten-stokes/elyra delete branch release-script

deleted time in 6 hours ago
pull request

kiersten-stokes pull request elyra-ai/elyra

kiersten-stokes
kiersten-stokes

Remove determination of absolute path value in KFP processor

When moving from storing the location attribute of Component objects as absolute rather than relative, this now-removed code block was made redundant. The behavior of os.path.join is such that this issue wasn't immediately obvious (see here).

What changes were proposed in this pull request?

Remove the unnecessary determination of absolute path value in kfp_processor.py.

How was this pull request tested?

One test has been updated to reflect these changes. Testing manually as well.

Developer's Certificate of Origin 1.1

   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.
pull request

kiersten-stokes pull request elyra-ai/elyra

kiersten-stokes
kiersten-stokes

Replace missing `metadata_class_name` in Component Registry schema

It was recently pointed out to me that the palette was not updating after changing a component registry as it should be after #2157. It looks like the metadata_class_name field was removed from the component registry schema in a subsequent PR.

What changes were proposed in this pull request?

Re-adds the metadata_class_name field to the component-registry schema

How was this pull request tested?

Unit tests are passing. Also tested manually by making changes to and deleting a registry and refreshing the JL page to confirm that the backend component registry cache is updating immediately after a change is made.

Developer's Certificate of Origin 1.1

   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.
Activity icon
created branch

kiersten-stokes in kiersten-stokes/elyra create branch registry-metadata-classname

createdAt 8 hours ago
Activity icon
created branch

kiersten-stokes in kiersten-stokes/elyra create branch remove-kfp-abspath

createdAt 8 hours ago
Oct
14
2 days ago
pull request

kiersten-stokes pull request ptitzler/elyra

kiersten-stokes
kiersten-stokes

Get KFP processor working end-to-end for MLX example

This might be useful for our discussion on Monday and as a companion example to the additions made to the BYO registry types discussion topic. Just a draft/jumping off point for now.

What changes were proposed in this pull request?

Added an optional attribute definition onto the Component object. If Component location_type evaluates to None (which is the case for the MLX example currently), the component definition will be saved to the Component object. During processing, location_type will again be evaluated, and text='component definition in text form' will be used as the argument to load_component() rather than url=... or filename=....

Also comments out a piece of unnecessary/redundant code, but I'll fix this with a proper PR later this week.

How was this pull request tested?

Developer's Certificate of Origin 1.1

   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.
Activity icon
created branch

kiersten-stokes in kiersten-stokes/elyra create branch byo-mlx-processor

createdAt 1 day ago
Activity icon
created branch

kiersten-stokes in kiersten-stokes/elyra create branch byo-mlx-cr-poc

createdAt 1 day ago
Oct
12
4 days ago
Activity icon
issue

kiersten-stokes issue elyra-ai/elyra

kiersten-stokes
kiersten-stokes

pipeline file: component description is stored in unexpected location

Describe the issue A clear and concise description of what the issue is.

The component description is stored as a child of component_parameters, which it is (conceptually) not.

        {
          "id": "855db8d9-6034-4e11-ae81-3cbff9782aa5",
          "type": "execution_node",
          "op": "download-file_DownloadFile2",
          "app_data": {
            "component_parameters": {
              "component_description": "Downloads a file from a public HTTP/S URL using a GET request.",
              "url": "https://raw.githubusercontent.com/ptitzler/examples/add-kfp-tutorial/pipelines/run-pipelines-on-kubeflow-pipelines/data/data.csv",
              "output_downloaded_file": ""
            },
            "label": "",
            "component_source": "https://raw.githubusercontent.com/elyra-ai/examples/master/pipelines/run-pipelines-on-kubeflow-pipelines/components/download-file.yaml",
            "ui_data": {

To Reproduce Steps to reproduce the behavior:

  1. Create a pipeline
  2. Add a custom component that includes a description
  3. Save the pipeline
  4. Open the .pipeline file in a text editor.

Expected behavior A clear and concise description of what you expected to happen.

Deployment information Describe what you've deployed and how:

  • Elyra version: 3.2/master
Activity icon
issue

kiersten-stokes issue comment elyra-ai/elyra

kiersten-stokes
kiersten-stokes

pipeline file: component description is stored in unexpected location

Describe the issue A clear and concise description of what the issue is.

The component description is stored as a child of component_parameters, which it is (conceptually) not.

        {
          "id": "855db8d9-6034-4e11-ae81-3cbff9782aa5",
          "type": "execution_node",
          "op": "download-file_DownloadFile2",
          "app_data": {
            "component_parameters": {
              "component_description": "Downloads a file from a public HTTP/S URL using a GET request.",
              "url": "https://raw.githubusercontent.com/ptitzler/examples/add-kfp-tutorial/pipelines/run-pipelines-on-kubeflow-pipelines/data/data.csv",
              "output_downloaded_file": ""
            },
            "label": "",
            "component_source": "https://raw.githubusercontent.com/elyra-ai/examples/master/pipelines/run-pipelines-on-kubeflow-pipelines/components/download-file.yaml",
            "ui_data": {

To Reproduce Steps to reproduce the behavior:

  1. Create a pipeline
  2. Add a custom component that includes a description
  3. Save the pipeline
  4. Open the .pipeline file in a text editor.

Expected behavior A clear and concise description of what you expected to happen.

Deployment information Describe what you've deployed and how:

  • Elyra version: 3.2/master
kiersten-stokes
kiersten-stokes

After discussing with @bourdakos1 earlier today, this is no longer an issue with new pipelines as of #2094

Activity icon
issue

kiersten-stokes issue comment elyra-ai/elyra

kiersten-stokes
kiersten-stokes

Explore support for 3rd party component registry/machine learning exchange

Is your feature request related to a problem? Please describe.

The LFAI Machine Learning Exchange project provides a component registry for Kubeflow Pipelines. We need to explore if the existing Elyra component registry implementation is flexible enough to support "proprietary" 3rd party component registries using a "plug and play" approach.

Describe the solution you'd like A clear and concise description of what you want to happen.

  • The Elyra registry should be able to pull components specifications from the 3rd party registry. (Currently only web resources accessible via anonymous GET requests are supported "registries")
  • Support for a specific 3rd party registry should perhaps be delivered by an optionally installable component ("plug and play") to enable developers to develop their own.
kiersten-stokes
kiersten-stokes

Discussion #2224 lays out how a BYO component registry type would look right now

Activity icon
created tag
createdAt 3 days ago
push

kiersten-stokes push elyra-ai/elyra

kiersten-stokes
kiersten-stokes

Update changelog for release 3.2.0

kiersten-stokes
kiersten-stokes

Prepare for next development iteration

kiersten-stokes
kiersten-stokes

commit sha: 2c3b1ab117eac6d9afd8750fdfff4eee9cdc06d4

push time in 3 days ago
Activity icon
created branch
createdAt 3 days ago
Previous