Skip to content

Conversation

benliang99
Copy link
Contributor

The Opacity plugin provides integration with the Opacity protocol for verifying AI inference proofs. It includes functionality for verifying proofs and interacting with Twitter for automated verification responses.

For a complete example implementation with an agent, see: https://github.com/SeraphAgent/ProofOfAutonomy.

@celesteanglm celesteanglm added the plugin Plugin contribution label Feb 8, 2025
@Ang-dot
Copy link
Contributor

Ang-dot commented Apr 29, 2025

Hey @benliang99, sorry to miss your PR out for some times, I'll be following-up on this. However, would need your help to rebase your changes to our latest branch and attach your PR according to our new PR template accessible via https://github.com/game-by-virtuals/game-python/blob/main/.github/PULL_REQUEST_TEMPLATE/plugin.md.

Thanks for the contribution!

@Ang-dot Ang-dot self-assigned this Apr 29, 2025
while True:
time.sleep(1)
except KeyboardInterrupt:
print("Shutting down...")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
print("Shutting down...")
print("Shutting down...")

Add one empty line EOF to comply with POSIX standard

if not verification.get("success"):
raise Exception("Proof is invalid")

return verification["success"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return verification["success"]
return verification["success"]

addopts = "-ra -q --cov=opacity_game_sdk"
testpaths = [
"tests",
]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
]
]

@@ -68,7 +68,7 @@ def __init__(self, options: Dict[str, Any]) -> None:
raise ValueError("Twitter API credentials are required.")

self.twitter_client: tweepy.Client = tweepy.Client(
bearer_token = credentials.get("bearerToken"),
Copy link
Contributor

Choose a reason for hiding this comment

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

this seems like an out of scope change, please remove, thanks!

@celesteanglm
Copy link
Contributor

Hi @benliang99 , gentle reminder on this - can you please look into incorporating the suggested changes?

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

Successfully merging this pull request may close these issues.

3 participants