I sort of think that adding FIXMEs is a good thing to do.  Adding this as an issue on github seems silly as this is unlikely to ever get fixed unless someone is actually cleaning up the code around it.

maybe we should decide on this sort of thing as a team.  In this particular instance this looks like we are doing unneeded work `key = key.encode('ascii', 'ignore')`  does nothing at all on the data I have.  Maybe it is needed but I cannot see where this happens.  This just looks like a dirty fix that was made at some point but there is no comment explaining why we are doing this, and it just adds complexity and slowness to indexing.

Personally I think adding FIXMEs is good as it will eventually lead to better code I hope.