-
Notifications
You must be signed in to change notification settings - Fork 666
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dev add spectral_norm #9674
base: master
Are you sure you want to change the base?
dev add spectral_norm #9674
Conversation
您好,之前看pytorch中的commit记录 module中增加相应模块的原因是为了 maintain BC
|
目前version是默认为1,所以是希望对标上这个hook;您的意思是对标version=0嘛,目前来看weight_norm似乎对标的是version=0, |
写SN目的还有一个是为了写SR的一些网络,我去实验看看SR那边需不需要保留version=1 |
好的~ |
其实也不是 就是 可以看下pytorch下面的两个hook 他们针对的是之前版本version=0的一个fix |
嗯嗯,我刚刚发现了我觉得是OK的,现在在看测试,目前测试会跟oneflow的Graph有一点冲突正在解决,稍后看麻烦review一下看看 |
嗯嗯 另外他后面version-1的in-place操作相关的fix主要是针对pytorch dataparallel master-slave模式带来的一些问题,oneflow的DistributedDataParallel和他的实现其实不一样。 我还没有跑SN的网络测试,不过感觉大概oneflow的模式大概能天然fix?单侧test里面感觉可以多加些ddp的~多交流 |
Speed stats:
|
View latest API docs preview at: https://staging.oneflow.info/docs/Oneflow-Inc/oneflow/pr/9674/ |
@BBuf 修复了一些 spectral_norm 实现过程中遇到的bug