Skip to content

Conversation

@d4l3k
Copy link
Member

@d4l3k d4l3k commented May 24, 2021

This creates a KFP adapter for the new specs interface.

import torchx.specs.api as torchx
from kfp import compiler, components, dsl

app = torchx.Application(...)
kfp_copy: Callable = component_from_app(app)

def pipeline() -> dsl.PipelineParam:
    a = kfp_copy()
    b = kfp_copy()
    b.after(a)
    return b

compiler.Compiler().compile(pipeline, "pipeline.zip")

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 24, 2021
@facebook-github-bot
Copy link
Contributor

@d4l3k has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@d4l3k has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@d4l3k has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Summary:
This creates a KFP adapter for the new specs interface.

```
import torchx.specs.api as torchx
from kfp import compiler, components, dsl

app = torchx.Application(...)
kfp_copy: Callable = component_from_app(app)

def pipeline() -> dsl.PipelineParam:
    a = kfp_copy()
    b = kfp_copy()
    b.after(a)
    return b

compiler.Compiler().compile(pipeline, "pipeline.zip")
```

Pull Request resolved: meta-pytorch/torchx#10

Differential Revision: D28658161

Pulled By: d4l3k

fbshipit-source-id: 33e344314ba5e1671c81f929e4a7e885b510e13a
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D28658161

facebook-github-bot pushed a commit that referenced this pull request May 25, 2021
…mpared to platform009's isort

Summary:
`isort==5.8.0` produces different sorting order compared to platform009's `isort` (not sure which version is used with platform009 but `isort==4.3.21` is the one we used to use for `pytorch/elastic` which produces consistent results to our internal `arc f`.

This should fix linter failures for both:
1. #12
2. #10

Reviewed By: d4l3k

Differential Revision: D28669450

fbshipit-source-id: 8ead84f96d2ef2d0c01ddaba2ce513c78f7966f3
@facebook-github-bot
Copy link
Contributor

@d4l3k merged this pull request in 5707d33.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants