https://developer.holochain.org/docs/tutorials/coreconcepts/hello_me/
I believe this:
fn retrieve_person
should be
pub fn retrieve_person
also, the span is named as address_output but then reference it as both an undeclared id variable and address. I think a once over on some of the variable naming would make the tutorial a bit easier to go through.
@rlkel0 thanks for letting us know! In the future if you see another issue you can click the āEditā link at the top and itāll create a GitHub issue against the page.
@freesig ping!
ok will do next time
1 Like
There seem to be a bunch of bugs in the Hello Me tutorial, maybe elsewhere as well - its hard as a newbie to Holochain to know whats really going on (logging is really verbose). I reported two of them, but after figuring around the smaller bugs in the other tutorials I hit Hello Me. I reported two of the bugs as @freesig suggested on Git, but the second just made it so blindingly obvious the tutorial was untested that I gave up ā¦
Iām hoping that someone who knows Holo could actually try one of the tutorials line for line and fix the pretty obvious discrepancies, then let us know so we can continue the learning process, rather than wasting hours trying to figure out what we are doing wrong, or understand the verbose debugging info, when its the tutorial itself that is broken.
1 Like
i actually did the tutorials recently. In the past there were bugs but now they run great for me. what was the bug you ran into?
A typo on ports used (its 9001 in one place and 9000 in the other), https://github.com/holochain/docs-pages/issues/153
and a complete mismatch between the āCheck your codeā box and the code entered above https://github.com/holochain/docs-pages/issues/154, I spent ages on the first one trying to interpret some very cryptic debugging before I noticed the comment about failing to connect, and gave up on the tutorials when I hit the discrepancies on the second.
I donāt think the second one is a bug, youāre saying the create_person check your code is too detailed? I think they just expose a couple extra utility functions to make it easier for you to expand on the example.
Hi @mitra42
Sorry to hear youāve had a bad experience. I appreciate the feedback. We do have automated testing and are running through the tutorials ourselves and with other people. Thereās obviously still gaps in this coverage though. I will schedule some time to look at these issues youāve mentioned and do a general sweep for problems.
Thereās also some further automated testing Iād like to add to address some of the things that have not been caught.
Hi Freesig, I donāt think automated testing is going to catch these - the issue is that you go through and add code following the tutorial. Then you get to the line āCheck your codeā and what Iāve written looks NOTHING like what pops up, because youāve added functions show_posts, show_person, show_output. (if youād said - weāve added a few more functions to speed the process up, then fine, but you didnāt and I donāt think you intended to, as I see now that you add these functions in a different form later in the tutorial) So as the learner, I go back and try and figure out what I did wrong. At this point I give up because clearly the tutorial writers arenāt checking their own work, and since Iāve already wasted a good hour looking for the bug I reported in #153, I figure why waste time on an obviously broken tutorial.
It looks to me that someone edited this page and left an old version of the code there.
Looks like similar lack of QC on the next āCheck Codeā where youāve got duplicate āshow_personā functions unless Iām missing something. One in the āZome callsā section and one in the āRender functionsā section, but not contained in any way, so looks like a bug to me.
Yep it most likely is a bug that I missed when checking. I think the issue is that the UI is not checked automatically and this is one of the things Iād like to add.
Sorry if this has caused you any frustration but thank you for bringing it to my attention.