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

Seldon graph complexity #3184

Closed
divyadilip91 opened this issue May 12, 2021 · 3 comments
Closed

Seldon graph complexity #3184

divyadilip91 opened this issue May 12, 2021 · 3 comments

Comments

@divyadilip91
Copy link

divyadilip91 commented May 12, 2021

image
Hi @RafalSkolasinski @cliveseldon . I am using seldon core 1.1.0 version .Can seldon graph be created for the above pipleine?
If so can you show how the seldon graph is structured.

@divyadilip91 divyadilip91 added the triage Needs to be triaged and prioritised accordingly label May 12, 2021
@divyadilip91
Copy link
Author

divyadilip91 commented May 12, 2021

graph: 
        children:
        - name: combiner1
          endpoint:
            type: REST
          type: COMBINER
          children:
          - name: model1
            endpoint:
              type: REST
            type: MODEL
            children: 
            - name: model2
              endpoint: 
                type: REST
              type: MODEL
              children: []
          - name: model1
            endpoint:
              type: REST
            type: MODEL
            children:
            - name: model3
              endpoint: 
                type: REST
              type: MODEL
              children: []
        name: outputtransformer
        endpoint:
          type: REST
        type: OUTPUT_TRANSFORMER

I tried the above graph and it does not seem to be working

@ukclivecox
Copy link
Contributor

For the current Seldon Core I think you can get it to work with

model1 -> output-transformer -> combiner -> (model2, model3)

You could also look at Tempo our new Python MLOps SDK which would make this structure much easier.

@ukclivecox ukclivecox removed the triage Needs to be triaged and prioritised accordingly label May 17, 2021
@divyadilip91
Copy link
Author

@cliveseldon Thankyou so much. The above graph you mentioned worked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants