Questions
**My questions and suggestions to improve the script.**
1. **Need more simple examples to motivate.** I suggest the authors do a simple experiments on how AdaDiag works on quadratic functions (with dense Hessian). This type of experiment will make AdaDiag more convincing.
2. **Extra computational overhead by SVD.** "While the total computational overhead induced by periodic SVD is negligible (less than 10%), " I kindly disagree this is negligible. Please add more discussions on the wall-clock running time and compare with AdamW rigorously. Right now, it is too hand-waving and informal.
3. **Extra memory.** Please write down the exact memory requirement of AdaDiag and compare to AdamW (e.g., 50% extra memory over Adam's optimizer state?). Please write down the exact memory footprint (in GB) on a concrete LLM architecture like Llama. Currently, Table 3 is too abstract and not so informative for the general audience.
4. **Distributed implementation.** Please discuss the challenges of implementing SVD when the weight matrix is shared in >=2 GPUs. Please discuss potential solutions.
5. **Ablation studies.** Please provide more ablation studies on the SVD frequency T. It would be interesting to see how the choice of T affects the performance.
4. What is the difference between AdaDiag and full-rank-Galore? Need more clarification.
To sum up, I believe the idea of "rotating the preconditioner" is reasonable and promising. The idea is worth sharing with the community. However, by judging the paper as a whole, I think the current script is still rather preliminary. For instance, the discussions on the weaknesses (e.g., memory and throughput) are too hand-waving and informal. I think it is okay to leave the memory & throughput issue for future investigation, but as a scientific paper, more serious discussion is needed to justify the current weaknesses.
Overall, I think this is a good paper with a very promising idea, but the current version needs more polishment to become a rigorous scientific paper.
**Some related works:** I suggest the authors add discussions on the following related works.
1. " first-order methods typically require meticulous tuning of hyperparameters to ensure the optimization process can converge to the desired local optima." The work [1] provides extensive evidence that hyperparameters (of Adam) need to be tuned carefully, otherwise the algorithm would diverge. Perhaps the authors can cite this work to ground the claim that " first-order methods typically require meticulous tuning ".
2. "these algorithms have demonstrated more robust empirical results than SGD in various domains and often exhibit better convergence behaviors in practice" The work [2,3] provides extensive evidence that Adam outperforms SGD on LLMs and investigates why it happens.
3. Section 2 in the work [4] also provides numerical evidence that the Adam preconditioner is more effective when the Hessian is rotated to a near-diagonal matrix. This also supports the effectiveness of the proposed AdaDiag (++).
[1] Zhang, Y., Chen, C., Shi, N., Sun, R., & Luo, Z. Q. (2022). Adam can converge without any modification on update rules.
[2] Kunstner, F., Yadav, R., Milligan, A., Schmidt, M., & Bietti, A. (2024). Heavy-tailed class imbalance and why adam outperforms gradient descent on language models.
[3] Zhang, Y., Chen, C., Ding, T., Li, Z., Sun, R., & Luo, Z. Q. (2024). Why transformers need adam: A hessian perspective.
[4] Zhang, Y., Chen, C., Li, Z., Ding, T., ... & Sun, R. (2024). Adam-mini: Use fewer learning rates to gain more.