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

Substrate - Feedback and local tweaks from UCB run-of-show incoperated #761

Open
4 of 5 tasks
Tracked by #708
nuke-web3 opened this issue Jul 24, 2023 · 4 comments
Open
4 of 5 tasks
Tracked by #708
Assignees

Comments

@nuke-web3
Copy link

nuke-web3 commented Jul 24, 2023

TODO before we close out the UCB cohort & tag the content repo

@JoshOrndorff
Copy link
Contributor

Regarding this FAQ about "why is consensus not part of the runtime?"
I think it is misleading to first give these weaker answers like "it is not often customized". Of course that is a true statement, but it gives the impression we could have put consensus in the runtime if we wanted to which is false. You only get to the real answer at the end which is that it is not possible to put consensus in your runtime; consensus is the tool that helps humans reach agreement about which history of the runtime is the real one. It is necessarily and fundamentally outside the runtime.

@nuke-web3
Copy link
Author

nuke-web3 commented Jul 24, 2023

we ask students to build substrate, I am assuming main - should we rather capture a version to be safe as we move through the Academy together? Perhaps a for just for us, tagged for each cohort as the "canonical" one we used.

Edit: indeed https://github.com/Polkadot-Blockchain-Academy/pba-ucb-frameles-node-template/pull/2 // https://github.com/Polkadot-Blockchain-Academy/frameless-node-template--master/pull/2 illustrates this well 🙃

@nuke-web3
Copy link
Author

See also: #764

@kianenigma
Copy link
Member

kianenigma commented Aug 9, 2023

Now that it is almost over, I briefly went over the student feedback and want to paste a few of them here:

Good:

It was so powerful to see the wasm blob upgrading as a storage entry on the database and getting demonstrated on Polkadot.js apps.

Bad: Two comments said that I oversimplify things, which is correct, and perhaps I do it too much.

Bad:

specifications were incomplete, unclear or even ambiguous and the assignment lacked proper test cases.

Middle:

The assignment, even though it was chaotic and the instructions ambiguous, it was a very effective way of learning and understanding a Substrate runtime

Good: Overall good feedback on the Client/Runtime/Wasm stuff, but that can always be improved:

Understanding the architecture of building a blockchain and how substrate in particular aims to do it in a modular, upgradable way.

Suggestion: Multiple people asked for Substrate/FRAME cheat sheet. Not sure what we can come up with, but a good idea.

Substrate Cheat Sheet about Storage and encoding

Suggestion: more people seem to want live coding. FRAMELess exercise could have this in it as well, if time permits.

Based comment in general:

  1. Implement a proper balance between theory and practice at the academy. 2. Ensure high quality of assignments and only provide them when they are ready. There is no excuse for Cambridgy or unclear assignments. 3. Do the FRAME lectures before the substrate lectures. 4. Abolish the substrate assignment and turn it into a blockchain-from-scratch style exercise to accompany substrate lectures. 5. Have a two week long FRAME assignment where students will have to implement a real use case in FRAME. This could be done either individually or in groups.

Bad: multiple people asked for consensus related content

I would've like to see the consensus part in the assignment or some hands-on exercise. I see pallets for consensus but currently have no idea where they go in substrate or FRAME.

Good: People in general liked the in-class FRAME testing + benchmarking + RPC exercises.

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

No branches or pull requests

5 participants