It seems good to me as well. It is hard to understand why didn't add any Ethereum or Litecoin to this? I mean both are useful, deserving coins to be added in many portfolio. I also think BNB should be less than %7.
I also don't know why i don't hold Ethereum, i just have BTC as main coin.
About the BNB:
I am holding BNB because i think it can be huge in the future (maybe like bittrex?), also they have some intersesting stuff coming up. Maybe you want to read this: