AIP-4600: [Part 1 of 3] Pull in v1.7.0 from upstream #7
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…w#5978 (kubeflow#6038) * chore(sdk): unify pipeline name between MLMD and KFP DB * backend changes pipeline-name during upload * update sdk snapshots * add backend tests * fix launcher * fix tests * address feedback
…ble mode. (kubeflow#6032) * Add submission time caching option override for v2 compatible mode. * fix tests * casting * disable cache for sample test * cleanup Co-authored-by: capri-xiyue <[email protected]>
* added v2 compatible cache e2e test * fixed typo * fixed e2e test * resued methods * combine cache_v2 and two_step pipeline * fixed sample v2 test * fixed cache e2e test * fixed e2e test * enabled cache e2e test * fixed typo * fixed typo * added enable caching info * added debug logging * fixed launcher bug * removed debug logging
…w#5930 (kubeflow#6039) * fix s3 artifact structure * Fix test
…onent_from_pipeline_func (kubeflow#5991)
* Components - Added HuggingFace dataset components * Added a sample pipeline
* Components - KFP - Run component This can be useful for dynamically running components or pipelines. * Added sample pipeline * Printing the run URL
…er (kubeflow#5919) * restructure the gcp component folder * update format
(kubeflow#6023) * feat(frontend): Clickable link css update * change color * Update clickable items styling * adjust execution list * fix test * fix test * fix test * implement row hover highlight for link
… too big. Fix kubeflow#6053 (kubeflow#6061) * quit using tfx library for visualization * revert to tensorflow * fix read file issue on FE
…flow#6060) * Components - Added canonical location for some components * Fixed the component.py files
…beflow#6064) * Adding Pytorch-lightning, mock and torchmetrics to tox.ini Signed-off-by: Shrinath Suresh <[email protected]> * Fixing lint issues Signed-off-by: Shrinath Suresh <[email protected]> * Bumping version to rc2 Signed-off-by: Shrinath Suresh <[email protected]> * Updating readme with the command to use tox Signed-off-by: Shrinath Suresh <[email protected]>
* Fix: Add sleep to cifar10 pipeline * Fix: Add comment for sleep and infernece tasks
kubeflow#6041) * Feature: Example to estimate best parameters using AX * fix: Change notebook path - fix voulme creation component * Fix: check if args has model params * fix: check model params in cifar10 script * Fix: Updated script for adam optimizer - Updated Readme and notebook with ax description * Fix: remove token from notebook * Fix: component yaml for trainer
* Feature: Bert distributed training * Feat: Adds staging volume for pytorch job * Feat: Add PVC storage URI for KFserving - Update copy component * Fix: gpu explain handler * fix: notebook cleanup * Fix: Update Dockerfile, requirement.txt * Fix: Dockerfile
…flow#6066) * add *.yaml files to python package * use a genrilized method to find yaml files. * Make the package_data path relative * reformat * use relative data path for yaml file.
kubeflow#6045) * Cloned to_gcs to experimental * Replaced GCSPath type with String * Switched GCPProject to String and fixed README * Added to_table component
…ython component to google_cloud_pipeline_components (kubeflow#6037) * restructure the gcp component folder * add dataflow component
…beflow#4715 (kubeflow#6072) * chore: integrate go-licenses in v2 launcher build process * fix * fix 2 * improve documentation * fix 3
* Add support for executor input * chagne method name to _executor_input_placeholder
…#6069) * Add target=_blank to cron format go.doc link * Use ExternalLink component to fix the cron link * Remove unnecessary whitespace Co-authored-by: Kelvin S. do Prado <[email protected]>
alexlatchford
changed the title
|
@alexlatchford do you have a recommendation on reviewing this? Was looking into your commit and it still has a ton of changes. I can just approve if this is simply pulling in changes upstream, but can you list some testing you did to validate this? |
Yeah effectively this is impossible to review 😅 Really the testing strategy has been to include this in all the example projects/integration tests etc. we have and also include it with all the other libraries we have. I'm not quite through all the tests yet but thus far it's looking pretty good! |
hsezhiyan
approved these changes
Sep 9, 2021
…an map to appropriate de|serializer (kubeflow#6504) Also: - Simplify _data_passing methods, add in type hints and docstrings. - Remove get_deserializer_code_for_type.
AIP-4600: [Part 3 of 3] Pull in upstream fixes from KFP.
AIP-4600: [Part 2 of 3] Add in publishing to ZG Artifactory
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Parts:
This PR just pulls in the KFP changes for the
v1.7.0tag and resolves the conflicts with the various changes we've made in teh ZG fork. All the changes ZG has made are still maintained!