Skip to content

Commit

Permalink
feat: new format for the controller metrics and operations
Browse files Browse the repository at this point in the history
Signed-off-by: Harikrishnan Balagopal <[email protected]>
  • Loading branch information
HarikrishnanBalagopal committed Apr 24, 2024
1 parent 8548a6d commit c68f06d
Show file tree
Hide file tree
Showing 17 changed files with 202 additions and 90 deletions.
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -36,3 +36,6 @@ venv/
*.bkp
*.bkp.*
*bkp*

# Auto generated file
tuning/_version.py
4 changes: 2 additions & 2 deletions examples/trainercontroller_configs/loss.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller
triggers:
Expand Down
3 changes: 2 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,8 @@ dependencies = [
"trl",
"peft>=0.8.0",
"datasets>=2.15.0",
"fire"
"fire",
"jsonschema>=4.21.1"
]

[project.optional-dependencies]
Expand Down
4 changes: 2 additions & 2 deletions tests/data/trainercontroller/loss_custom_metric.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
testflag:
CustomMetric:
- name: testflag
class: CustomMetric
controllers:
- name: loss-controller-custom-metric
triggers:
Expand Down
8 changes: 4 additions & 4 deletions tests/data/trainercontroller/loss_custom_operation.yaml
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
operations:
customoperation:
CustomOperation:
- name: customoperation
class: CustomOperation
controllers:
- name: loss-controller-custom-operation
triggers:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
operations:
customoperation:
CustomOperationInvalidAction:
- name: customoperation
class: CustomOperationInvalidAction
controllers:
- name: loss-controller-custom-operation-invalid-action
triggers:
Expand Down
4 changes: 2 additions & 2 deletions tests/data/trainercontroller/loss_invalid_metric.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
MissingMetricClass:
- name: loss
class: MissingMetricClass
controllers:
- name: loss-controller-invalid-metric
triggers:
Expand Down
4 changes: 2 additions & 2 deletions tests/data/trainercontroller/loss_invalid_operation.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller-invalid-operation
triggers:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller-invalid-operation-action
triggers:
Expand Down
4 changes: 2 additions & 2 deletions tests/data/trainercontroller/loss_invalid_trigger.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller-invalid-trigger
triggers:
Expand Down
4 changes: 2 additions & 2 deletions tests/data/trainercontroller/loss_on_threshold.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller
triggers:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller-wrong-input-rule
triggers:
Expand Down
4 changes: 2 additions & 2 deletions tests/data/trainercontroller/loss_with_malicious_os_rule.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
controller-metrics:
loss:
Loss:
- name: loss
class: Loss
controllers:
- name: loss-controller-wrong-os-rule
triggers:
Expand Down
Loading

0 comments on commit c68f06d

Please sign in to comment.