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

Height and width confusion in show dimensions text using AlexNet webGL #52

Open
sumeyye-agac opened this issue Jan 14, 2023 · 2 comments

Comments

@sumeyye-agac
Copy link

Tensor width and height order seems to to be incorrect. (For filter size dimensions, it is not wrong)

Screen Shot 2023-01-14 at 23 24 59

@alexlenail
Copy link
Owner

@sumeyye-agac thanks for reporting this. Would changing the order of the help text (right underneath the word architecture be enough to fix this?

@jpauwels
Copy link
Contributor

Hi @alexlenail. The text underneath "Architecture" is correct, as you can see that a height of 50 and a width of 500 gives rectangles that are wider than they are tall, but the labels are simply misplaced. The 50 label should be put on the height, not the width. Fixed in PR #60.

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

3 participants