Skip to content
This repository was archived by the owner on Sep 9, 2025. It is now read-only.

Conversation

@russellb
Copy link
Member

The design proposal in #109, and the corresponding implementation in
instructlab/sdg#86, raised the importance of
clearly defining how a custom pipeline that requires a model with
custom adapters would be configured. This document explores that
topic.

It's possible this should just become a subsection of #109.

Signed-off-by: Russell Bryant rbryant@redhat.com

The design proposal in instructlab#109, and the corresponding implementation in
instructlab/sdg#86, raised the importance of
clearly defining how a custom pipeline that requires a model with
custom adapters would be configured. This document explores that
topic.

It's possible this should just become a subsection of instructlab#109.

Signed-off-by: Russell Bryant <rbryant@redhat.com>
@russellb russellb force-pushed the sdg-serving-config branch from a72e0f2 to b4d3fec Compare July 10, 2024 19:42
Comment on lines +65 to +70
backend_args:
vllm_args:
- ...
- "--lora-modules"
- "my_custom_adapter=path/to/my_custom_adapter"
- ...
Copy link
Member Author

Choose a reason for hiding this comment

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

possible impact to this syntax here: instructlab/instructlab#1635

Copy link
Member Author

Choose a reason for hiding this comment

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

version: "1.0"
models:
- name: my_custom_adapter
description: a funky adaptor for generating questions
Copy link
Contributor

Choose a reason for hiding this comment

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

The models section really isn't necessary in retrospect. This will work:

version: "1.0"
models:
  - name: myfunkyadaptor
    description: a funky adaptor for generating questions
block_configs:
  - block_type: LLMBlock
    block_config:
      block_name: gen_questions
      config_path: configs/skills/freeform_questions.yaml
      add_num_samples: True
      gen_kwargs:
        model_id: myfunkyadatpor
      output_cols:
        - question
    drop_duplicates:
      - question

And then the serving config just needs to have --lora-modules=myfunkyadapter=path/to/my_custom_adapter

@russellb
Copy link
Member Author

The important details from here are now integrated into #109

@russellb russellb closed this Jul 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants