It’s a bit hard to understand when the screen shot has no node titles or information pinned or warnings shown or mentioned.
If you haven’t done so, I would suggest reading How to get help on the Dynamo forums
Start by looking at the list levels of your List.GetItemAtIndex nodes
You are using the ElementId value of the family
From the API documentation
ids are subject to change during an Autodesk Revit session and as such should not be retained and used across repeated calls to external commands. If a manner is needed to uniquely identify an element beyond this limitation then a shared parameter should be added to the element containing a unique identifier managed by the external application.
You can use the elements UniqueId property and fetch with ByUniqueId from Crumple nodes
Next element locations can be points or curves - it may be easier to use Element.MoveByVector