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

Add MazzumaGPT to Builder Tools #1111

Merged
merged 11 commits into from Aug 11, 2023
Merged

Conversation

StarkOsae
Copy link
Contributor

@StarkOsae StarkOsae commented Jul 25, 2023

Checklist

Builder Tool addition

@rphair rphair added the builder tool Indicates a PR/issue on a builder tool label Jul 25, 2023
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

Very interesting idea; we don't have anything like it on the Developer Portal yet.

Right now the only mention of Cardano I can find on your site is in a footnote which you have to accordion out from the FAQ on the front page, where it mentions Plutus as a second item after Solidity. Do you have anything we might include as a getstarted: link which documents how your system works, both generally and particularly for Cardano & Plutus?

In fact, do you have any page that fulfills a documentation purpose? (The way it looks today, I have to "create an account" to see anything beyond the front page.) If so you can add it to getstarted:... but without it, it wouldn't feel safe to me as a user to use it, or as a reviewer to endorse it.

Also you have to remove your yarn.lock file from your PR, which can be done by following the instructions here: https://developers.cardano.org/docs/portal-contribute#i-committed-yarnlock-to-my-pr-branch-what-do-i-do

@rphair rphair changed the title Added MazzumaGPT to Builder Tools Add MazzumaGPT to Builder Tools Jul 25, 2023
@StarkOsae
Copy link
Contributor Author

Very interesting idea; we don't have anything like it on the Developer Portal yet.

Right now the only mention of Cardano I can find on your site is in a footnote which you have to accordion out from the FAQ on the front page, where it mentions Plutus as a second item after Solidity. Do you have anything we might include as a getstarted: link which documents how your system works, both generally and particularly for Cardano & Plutus?

In fact, do you have any page that fulfills a documentation purpose? (The way it looks today, I have to "create an account" to see anything beyond the front page.) If so you can add it to getstarted:... but without it, it wouldn't feel safe to me as a user to use it, or as a reviewer to endorse it.

Also you have to remove your yarn.lock file from your PR, which can be done by following the instructions here: https://developers.cardano.org/docs/portal-contribute#i-committed-yarnlock-to-my-pr-branch-what-do-i-do

Thanks for the feedback.
I've added the link to the project's documentation page and also removed the yarn.lock file

@rphair
Copy link
Collaborator

rphair commented Jul 25, 2023

thanks @StarkOsae - though the documentation seems to confirm this is Ethereum centred, with all the examples being given for Solidity. Since the documentation doesn't explain much of how the system works, or what the expectations of Cardano / Plutus users would be (particularly regarding security or verification of smart contracts), I'd have to follow the feedback of other reviewers who work more with smart contract programming.

In the meantime, you may have removed yarn.lock from some environment you are using, but it's still present in the branch for this pull request. One again I would suggest you try the instructions that the link I sent you above, which will work. When you've removed it properly you'll see that yarn.lock will be gone from this page: https://github.com/cardano-foundation/developer-portal/pull/1111/files

@StarkOsae
Copy link
Contributor Author

thanks @StarkOsae - though the documentation seems to confirm this is Ethereum centred, with all the examples being given for Solidity. Since the documentation doesn't explain much of how the system works, or what the expectations of Cardano / Plutus users would be (particularly regarding security or verification of smart contracts), I'd have to follow the feedback of other reviewers who work more with smart contract programming.

In the meantime, you may have removed yarn.lock from some environment you are using, but it's still present in the branch for this pull request. One again I would suggest you try the instructions that the link I sent you above, which will work. When you've removed it properly you'll see that yarn.lock will be gone from this page: https://github.com/cardano-foundation/developer-portal/pull/1111/files

Thanks for the recommendations.
As suggested, we have added Plutus examples in the documentation to make the application more user-friendly for the Cardano community.
Also, the yarn.lock has been removed and is no longer in the pull request as you directed.

@rphair
Copy link
Collaborator

rphair commented Jul 30, 2023

@rdlrt @katomm @os11k @fill-the-fill @gitmachtl how could we possibly go about vetting something like this? There is some building interest in this field (e.g. https://forum.cardano.org/t/nlp-driven-smart-contracts-and-knowledge-tools/120224) but how do we know something built for Solidity just hasn't been resurfaced for Plutus? And how much would this matter? 🤔

@StarkOsae
Copy link
Contributor Author

@rdlrt @katomm @os11k @fill-the-fill @gitmachtl how could we possibly go about vetting something like this? There is some building interest in this field (e.g. https://forum.cardano.org/t/nlp-driven-smart-contracts-and-knowledge-tools/120224) but how do we know something built for Solidity just hasn't been resurfaced for Plutus? And how much would this matter? 🤔

@rphair In building MazzumaGPT, we curated the Plutus dataset from the Plutus Pioneer Program. A technical team from Cardano Foundation was involved in advising us on which Plutus code modules should be included in our custom LLM. Even though the model does Solidity as well, a lot of testing and code reviews has been done by the Cardano Foundation on our model's Plutus capabilities before we released it to the public. You can check out our Plutus prompt library (https://mazzumagpt.gitbook.io/mazzumagpt-docs/mazzumagpt-prompt-library#plutus-sample-prompts) for example use cases that MazzumaGPT supports.

Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

@StarkOsae that's good enough for me. This needs approval from 2 other reviewers who will all bring their own point of view & I'll tag them to have a look at this. 😎

@rphair rphair merged commit 138b967 into cardano-foundation:staging Aug 11, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builder tool Indicates a PR/issue on a builder tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants