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

The order of the input parameters of the AFB2D function #33

Closed
wdanc opened this issue Aug 25, 2021 · 2 comments
Closed

The order of the input parameters of the AFB2D function #33

wdanc opened this issue Aug 25, 2021 · 2 comments

Comments

@wdanc
Copy link

wdanc commented Aug 25, 2021

Thanks for your great work.
I have a question about the order of the input parameters of the AFB2D function. In the transform2d.py line 70-71, the function take input parameters as
ll, high = lowlevel.AFB2D.apply(ll, self.h0_col, self.h1_col, self.h0_row, self.h1_row, mode)

ll, high = lowlevel.AFB2D.apply(

however, the function AFB2D forwardd in lowlevel.py line 336 take the order of inputs as h0_row, h1_row, h0_col, h1_col

def forward(ctx, x, h0_row, h1_row, h0_col, h1_col, mode):

so, the order of the input "col col row row" seems to be inconsistent with the function's definition "row row col col". I wonder why it is entered in this way?

@fbcotter
Copy link
Owner

No reason, just bad practice by me! Happy to change it to be consistent

@wdanc
Copy link
Author

wdanc commented Oct 28, 2021

No reason, just bad practice by me! Happy to change it to be consistent

Thanks for your reply. I have read relative functions, and I noticed you make sure the filters have the right dimension, therefore, the input order won't impact the result. And I'm happy that your answer helps me to confirm that.

@wdanc wdanc closed this as completed Oct 28, 2021
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

2 participants