Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Interview the internal auditor to see what kind of specification they will require to audit the protocol logic #502

Closed
Tracked by #448
ch1bo opened this issue Sep 20, 2022 · 2 comments
Assignees
Labels
task Subtask of a bigger feature.

Comments

@ch1bo
Copy link
Collaborator

ch1bo commented Sep 20, 2022

No description provided.

@ch1bo ch1bo added the task Subtask of a bigger feature. label Sep 20, 2022
@ch1bo ch1bo self-assigned this Sep 20, 2022
@ch1bo
Copy link
Collaborator Author

ch1bo commented Sep 23, 2022

From latest interaction

  • The new spec "looks simpler" than the paper. (Although not compared in detail I guess)
  • Would benefit from list of interactions / use cases
  • After chatting a bit, a state diagram explaining the interaction (individual API messages + side effects) with the hydra-node could help .. maybe also a dedicated architecture page.

@ch1bo ch1bo assigned ffakenz and unassigned ch1bo Nov 11, 2022
@ffakenz
Copy link
Contributor

ffakenz commented Nov 11, 2022

During the last interview with the auditor (Yun Lu), we went through the spec document to resolve questions and provide clarifications.

Observations:

  • It was mentioned by @arnaud that it would be great to have, as an outcome, a specification anyone can follow to implement the hydra-head protocol.
  • Yun mentioned that she is currently working on getting a high-level overview of the spec, and providing some starting questions over email, and that she was going to start getting into the details later on. We agree not to continue the discussions over email but over the spec document or Slack.
  • @arnaud shared the Miro board "head full life cycle," which the auditor found useful (an updated version will be sent to her).
  • There were some things unclear to Yun:
    • Why not define "val" assets using Set notation?
    • What exactly is the PT and how does it work (what role does it play in the protocol)?
  • Yun's questions revealed her lack of experience with Plutus scripts for writing smart contracts.
  • Yun mentioned she only has a fixed box time of 2 hours per week to work on the spec.
  • We only covered the "on-chain validators" section of the spec document during this interview.

@ch1bo ch1bo closed this as completed Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Subtask of a bigger feature.
Projects
None yet
Development

No branches or pull requests

2 participants