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

Add parent-sha input #257

Merged
merged 5 commits into from
Sep 27, 2023
Merged

Add parent-sha input #257

merged 5 commits into from
Sep 27, 2023

Conversation

daavoo
Copy link
Contributor

@daavoo daavoo commented Sep 25, 2023

@daavoo daavoo added the A: example-get-started-experiments DVC Experiment, DVCLive examples label Sep 25, 2023
@daavoo daavoo self-assigned this Sep 25, 2023
@daavoo daavoo temporarily deployed to aws September 26, 2023 09:18 — with GitHub Actions Inactive
@daavoo daavoo temporarily deployed to aws September 26, 2023 09:21 — with GitHub Actions Inactive
@daavoo daavoo temporarily deployed to aws September 26, 2023 10:23 — with GitHub Actions Inactive
@daavoo daavoo temporarily deployed to aws September 26, 2023 10:25 — with GitHub Actions Inactive
@daavoo daavoo marked this pull request as ready for review September 26, 2023 10:51
@daavoo daavoo temporarily deployed to aws September 26, 2023 10:59 — with GitHub Actions Inactive
@daavoo daavoo temporarily deployed to aws September 26, 2023 11:01 — with GitHub Actions Inactive
tags:
- "results/train=pool-segmentation#*"
- "*"
Copy link
Contributor Author

@daavoo daavoo Sep 26, 2023

Choose a reason for hiding this comment

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

This was dependent on a hardcoded model name and it got broken after the prefix was removed.
It should be okay to trigger on all tags as there is a check below that only tries to deploy if gto parses the tag as assigment (I assume it would not that for unrelated tags)

Choose a reason for hiding this comment

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

Should be okay, but in theory we need to account for repos with multiple models, right? What do you think is best? Should we filter here or parse info from gto below?

@daavoo daavoo temporarily deployed to aws September 26, 2023 11:49 — with GitHub Actions Inactive
@daavoo daavoo temporarily deployed to aws September 26, 2023 11:51 — with GitHub Actions Inactive

runner-job:
needs: deploy-runner
runs-on: [ self-hosted, cml ]
environment: cloud
container:
image: iterativeai/cml:0-dvc2-base1-gpu
image: iterativeai/cml:latest-gpu

Choose a reason for hiding this comment

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

Is this working or broken?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Both are broken 😢

Copy link

@dberenbaum dberenbaum left a comment

Choose a reason for hiding this comment

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

Seems unfortunate that GH requires this to trigger a workflow for a non-head ref, but LGTM

@daavoo daavoo merged commit ecd916e into master Sep 27, 2023
2 checks passed
@daavoo daavoo deleted the parent-sha branch September 27, 2023 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: example-get-started-experiments DVC Experiment, DVCLive examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants