Skip to content
Discussion options

You must be logged in to vote

Hi @Khoa-NT, thanks for your question.

A fastMRI sample should use the 2D format with C being the number of slices.

In my knownledge, you mean reshape the 3D volume from (B,C,W,H,D) to (B,C*D,W,H) ?

If you're working on the fastMRI dataset, then no need to reshape because fastMRI's default shape is (B,num_slices,W,H).

However, the current code only support C=1, therefore, the loss can't run with (B,C*D,W,H).

Good point, since there are datasets where C varies across samples, the current implementation works with C=1 and the loss function should be called for each channel, separately. We'll correct this in the docstring.

Does this address your issue?

Replies: 2 comments

Comment options

You must be logged in to vote
0 replies
Answer selected by Khoa-NT
Comment options

You must be logged in to vote
0 replies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Q&A
Labels
None yet
3 participants