This is fine for ngrams of size 1 where as a lot of entities are two even three tokens. Your training set needs to include different ngram sizes or this is very limited, especially when you’re training your own custom entities. You wouldn’t want to go through this hassle when there are dozens of out of the box NER models pretrained on the usual entities.