Skip to content

Task validator filter#112

Merged
p-ferreira merged 9 commits intostagingfrom
features/task-validator-filter
Aug 2, 2023
Merged

Task validator filter#112
p-ferreira merged 9 commits intostagingfrom
features/task-validator-filter

Conversation

@p-ferreira
Copy link
Contributor

@p-ferreira p-ferreira commented Jul 31, 2023

Adds task validator filter to check if the proposed task is being accomplished.

This PR seeks to bring more consistency to the CoT developed by the validators and the miners, preventing tasks to be performed out of their prompts.

This is an initial simple keyword check verification model that can be evolved to more sophisticated use cases with real model implementation.

  • adds --neuron.task_validator_off flag
  • adds task validator model
  • updates event structure to support task validator
  • adds unit tests for task validator and updates event tests
  • includes and complements prompt engineering work from fixing spelling #110

@p-ferreira p-ferreira changed the title Task validator filter Task validator filter [WIP] Jul 31, 2023
@mrseeker
Copy link
Contributor

This can easily be bypassed by avoiding to mention "Question:". You might need to alter it to lowercase only and rethink the way questions are asked. The AI could also use "reason" or "thought" or something in those lines, or even avoid the words Question or Answer by giving an alternative word with a similar meaning.

@p-ferreira
Copy link
Contributor Author

This can easily be bypassed by avoiding to mention "Question:". You might need to alter it to lowercase only and rethink the way questions are asked. The AI could also use "reason" or "thought" or something in those lines, or even avoid the words Question or Answer by giving an alternative word with a similar meaning.

Well noted, I implemented the verification to be case insensitive. I totally agree with you, as the network evolves the attacks will also evolve and we will need more sophisticated ways to validate if the completion is related to the task being proposed.

As of now, this simple verification should take care of good part of the current "topic drifts" that we see popping up in the network, we will follow this issue closely to what's being generated in the network

@p-ferreira p-ferreira marked this pull request as ready for review July 31, 2023 21:49
@p-ferreira p-ferreira changed the title Task validator filter [WIP] Task validator filter Jul 31, 2023
@p-ferreira p-ferreira requested review from Eugene-hu and isabella618033 and removed request for Eugene-hu July 31, 2023 22:59
Copy link
Contributor

@camfairchild camfairchild left a comment

Choose a reason for hiding this comment

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

LGTM

@p-ferreira p-ferreira mentioned this pull request Aug 1, 2023
@p-ferreira p-ferreira merged commit 10cb6de into staging Aug 2, 2023
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.

4 participants