method naming
param naming
pep8
we already have the node entity, don't we?