Skip to content

Conversation

@xiaochulaoban
Copy link

Added the README and script files for training sql_agent on NPU

@xiaochulaoban
Copy link
Author

/cla

@xiaochulaoban
Copy link
Author

@microsoft-github-policy-service agree company="HUAWEI"

@@ -0,0 +1,200 @@
"""Train an SQL agent on the Spider dataset using Agent-lightning.
Copy link
Contributor

Choose a reason for hiding this comment

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

what's the difference of this new script and the old script?

Copy link
Author

Choose a reason for hiding this comment

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

Added configuration for training on NPUs.

Copy link
Contributor

Choose a reason for hiding this comment

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

which is?

Copy link
Author

Choose a reason for hiding this comment

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

Added "device": "npu" to the trainer and modified several other parameters.

Copy link
Contributor

Choose a reason for hiding this comment

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

In that case, create a new function in like config_train_npu in the original script and add a section in the README will be enough. You can reference this README in the spider recipe.

Copy link
Contributor

Choose a reason for hiding this comment

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

We don't want to include a dedicated script in a CI-maintained examples for some hardwares that is not included in the CI., unless we will have a NPU machine for daily testing.

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

Successfully merging this pull request may close these issues.

2 participants