# Learn Algorithm Design by Building a Shortest Path Algorithm - Step 21

### Tell us what’s happening:

Describe your issue in detail here.

how to solve this ?

``````my_graph = {
'A': [('B', 3), ('D', 1)],
'B': [('A', 3), ('C', 4)],
'C': [('B', 4), ('D', 7)],
'D': [('A', 1), ('C', 7)]
}

/* User Editable Region */

def shortest_path(graph, start):
unvisited = []
for cat in my_graph:
unvisited.append(cat)

/* User Editable Region */

``````

User Agent is: `Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36`

### Challenge Information:

Learn Algorithm Design by Building a Shortest Path Algorithm - Step 21

You appear to have created this post without editing the template. Please edit your post to Tell us what’s happening in your own words.

Hello Rahulp, welcome to the forum!

The question asked you to loop over `graph` (the variable that is given to you by the function signature).
Right now you are looping over `my_graph` instead.

Also, although it’s not required, I would advise calling the variable you use something that makes more sense so it wont get confusing. You’re not getting cats from the graph

1 Like

This is not the right name. See what the function accepts as parameter.

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.