Salvo los transistores, todo bien. Los transistores que sean un BD139 por ejemplo, u otro de más corriente, por lo menos 1 ó 2A, ya que como tienes 24 columnas, y tus leds digamos que cada uno a 25mA consumirás 600mA con toda una fila encendida, y un BC557 se queda corto, puede que caliente mucho, se queme, o que no tengas un brillo bueno.
No te olvides de los condensadores de desacoplo de unos 100nF en el micro y en cada integrado.
saludos.