Skip to content
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

新版本的遇到的疑问 #103

Open
chasingone opened this issue May 11, 2023 · 6 comments
Open

新版本的遇到的疑问 #103

chasingone opened this issue May 11, 2023 · 6 comments

Comments

@chasingone
Copy link

作者您好,在老版本的MedSegDiff上面是使用了FF的,为什么你重新修改后改写的UNet中并没有FF变换,这是因为原来的效果不好吗?那么这和论文就不对应了,请问这是什么原因?

@MurphyYe
Copy link

Yeah, I also have the same problem like u. And the other question is that in the paper said that in attention mechanism it will be used layer norm but I found that actually it is instance norm.

@goghfan
Copy link

goghfan commented Jan 2, 2024

+1 我也发现了上述两个问题,在新版本中默认去除了FFParse,这跟medsegdiff v2的论文流程图对应不起来,想知道是什么原因。

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


+1 I also discovered the above two problems. FFParse is removed by default in the new version. This does not correspond to the paper flow chart of medsegdiff v2. I want to know the reason.

@cannonli7
Copy link

cannonli7 commented Mar 20, 2024

+1 我也发现了上述两个问题,在新版本中默认去除了FFParse,这跟medsegdiff v2的论文流程图对应不起来,想知道是什么原因。

你好,我发现v2版本的ssformer好像在代码里没有?v2论文里的ssformer的输入有两个,然后代码里关于qkv的attention模块都是单输入的?请问已找到ssformer模块在哪里了吗?

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


+1 I also discovered the above two problems. FFParse is removed by default in the new version. This does not correspond to the paper flow chart of medsegdiff v2. I want to know the reason.

Hello, I found that the v2 version of ssformer does not seem to be in the code? The ssformer in the v2 paper has two inputs, and the attention module about qkv in the code has a single input?

@SiHengHeHSH
Copy link

I have the same problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants