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.

Sign up for GitHub

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

Jump to bottom

Handle different base paths explorer #427

Merged
merged 5 commits into from
Sep 18, 2023
Merged

Handle different base paths explorer #427

merged 5 commits into from
Sep 18, 2023

Commits on Sep 11, 2023

  1. handle different base paths explorer

    @PhilippeMoussalli
    PhilippeMoussalli committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    de1b668 View commit details
    Browse the repository at this point in the history

Commits on Sep 12, 2023

  1. update strategy to always pass base path

    @PhilippeMoussalli
    PhilippeMoussalli committed Sep 12, 2023
    Configuration menu
    Copy the full SHA
    02b2710 View commit details
    Browse the repository at this point in the history

Commits on Sep 15, 2023

  1. remove extra fs file

    @PhilippeMoussalli
    PhilippeMoussalli committed Sep 15, 2023
    Configuration menu
    Copy the full SHA
    d6f4161 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into data-explorer-handle-base-paths

    @PhilippeMoussalli
    PhilippeMoussalli committed Sep 15, 2023
    Configuration menu
    Copy the full SHA
    2a67811 View commit details
    Browse the repository at this point in the history

  3. Update data explorer interface (#431) 

    First review  #427

PR that updates the interface of the data explorer 

![image](https://github.com/ml6team/fondant/assets/47530815/964f640a-af24-46a2-b595-e1a24f168ab1)

Notes: 
* Right now the base path that we currently have has a mix of subsets
and fields. Related to #244
* The current base paths contains a mix of subsets and pipelines due to
the old way of organizing. In the future it should only include pipeline
names.
#368 (comment)
Perhaps we can do some cleanup to our current base path and remove
unnecessary subsets

```
gs://soy-audio-379412_kfp-artifacts/custom_artifact
```
    @PhilippeMoussalli
    PhilippeMoussalli authored Sep 15, 2023
    Configuration menu
    Copy the full SHA
    b8e20ff View commit details
    Browse the repository at this point in the history