Several popular ERC20 tokens are incompatible with the vault due to MAX approve
There are several instances where the vault approves use of funds to the manager or a trade router. It will set approval to MAX_UINT256.\\n```\\n         for (uint i = 0; i < tokens.length; i++) {\\n         //allow vault manager to withdraw tokens\\n                   IERC20(tokens[i]).safeIncreaseAllowance(ownerIn, \\n         type(uint256).max); \\n         }\\n```\\n\\nThe issue is that there are several popular tokens(https://github.com/d-xo/weird-erc20#revert-on-large-approvals--transfers) (UNI, COMP and others) which do not support allowances of above UINT_96. The contract will not be able to interoperate with them.
Consider setting allowance to UINT_96. Whenever the allowance is consumed, perform re-approval up to UINT_96.
```\\n         for (uint i = 0; i < tokens.length; i++) {\\n         //allow vault manager to withdraw tokens\\n                   IERC20(tokens[i]).safeIncreaseAllowance(ownerIn, \\n         type(uint256).max); \\n         }\\n```\\n