我刚刚开始测试uBLAS pr #68,该请求要求将thomasyang1207:develop从GSOC 2015合并到uBLAS。为此,我使用的是new branch中的GluCat。
我正在使用GluCat的test11进行测试,该方法广泛使用先验函数,其中一些函数涉及矩阵对数和平方根,而平方根又调用了特征求解器来确定矩阵的(复杂)特征值。以我的经验,本征分解器第一次被赋予4 x 4矩阵时,测试变得缓慢得令人无法接受。我还没有诊断出问题,但是可能代码复制的矩阵超出了实际需要吗?
也。我已经看过代码本身,并且有很多注释。在这个问题中,我仅要陈述一个:代码的读取方式就像使用“ complex”一词来表示“虚数”一样,就像将复数分为实数部分和“ complex”部分一样。如果要保留在代码的公共界面中,这将是不必要的混乱原因,并且实际上,只要正确且适当,在整个代码中,“复杂”都应更改为“ imag”或“虚构”所以。
我还有更多评论,但是如何通过在uBLAS存储库中对uBLAS pr #68进行代码审查来做出评论?谁必须授予我访问权限?我有C ++以及线性和多线性代数的背景知识,并且在ANU 2009-2011上教过数值线性代数,因此,只要有时间,我就可以胜任这项工作。
答案 0 :(得分:0)
感谢您考虑Boost/numeric/ublas并测试了拉取请求uBLAS pr #68。我们仍在将以前的Google Summer of Code项目纳入我们的开发部门的进度。
我还没有诊断出问题,但是可能代码复制的矩阵超出了实际需要吗?
请求的拉取尚未经过足够的测试和基准测试。因此,我们感谢能获得的所有帮助。
我还有更多评论,但是如何通过在uBLAS存储库中对uBLAS pr #68进行代码审查来做出评论?
如果您直接在github pull request页面uBLAS pr #68中发表评论,我们将非常高兴。请随时加入我们的gitter channel。
最佳 Cem