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

fix images render #743

Merged
merged 2 commits into from
Jul 19, 2022
Merged

fix images render #743

merged 2 commits into from
Jul 19, 2022

Conversation

spirosChv
Copy link
Collaborator

resolves #739

@github-actions
Copy link

W2D5_Tutorial2
Instructor Open In Colab
Student Open In Colab
Code report for W2D5_Tutorial2

Code report for W2D5_Tutorial2.ipynb

Quality (pyflakes)

Total code issues: 39

Cell 6, Line 10: redefinition of unused 'random' from line 25
Cell 6, Line 11: redefinition of unused 'torch' from line 24
Cell 9, Line 2: redefinition of unused 'widgets' from line 40
Cell 9, Line 15: undefined name 'display'
Cell 9, Line 22: undefined name 'display'
Cell 9, Line 31: undefined name 'display'
Cell 10, Line 15: undefined name 'display'
Cell 10, Line 22: undefined name 'display'
Cell 10, Line 31: undefined name 'display'
Cell 15, Line 15: undefined name 'display'
Cell 15, Line 22: undefined name 'display'
Cell 15, Line 31: undefined name 'display'
Cell 17, Line 2: redefinition of unused 'TextCNN' from line 445
Cell 20, Line 2: redefinition of unused 'VanillaRNN' from line 542
Cell 22, Line 15: undefined name 'display'
Cell 22, Line 22: undefined name 'display'
Cell 22, Line 31: undefined name 'display'
Cell 23, Line 15: undefined name 'display'
Cell 23, Line 22: undefined name 'display'
Cell 23, Line 31: undefined name 'display'
Cell 27, Line 2: redefinition of unused 'evaluate' from line 757
Cell 29, Line 15: undefined name 'display'
Cell 29, Line 22: undefined name 'display'
Cell 29, Line 31: undefined name 'display'
Cell 32, Line 2: redefinition of unused 'TaggingRNN' from line 975
Cell 35, Line 15: undefined name 'display'
Cell 35, Line 22: undefined name 'display'
Cell 35, Line 31: undefined name 'display'
Cell 36, Line 3: 'io' imported but unused
Cell 40, Line 53: local variable 'context' is assigned to but never used
Cell 41, Line 2: redefinition of unused 'Encoder' from line 1315
Cell 41, Line 24: redefinition of unused 'Decoder' from line 1343
Cell 47, Line 14: local variable 'attention_weight_seq' is assigned to but never used
Cell 49, Line 15: undefined name 'display'
Cell 49, Line 22: undefined name 'display'
Cell 49, Line 31: undefined name 'display'
Cell 55, Line 24: local variable 'output_dist' is assigned to but never used
Cell 56, Line 2: redefinition of unused 'evaluateMultinomial' from line 1710
Cell 57, Line 3: redefinition of unused 'grad_clipping' from line 841

Style (pycodestyle)

Total PEP8 violations: 162

Common problems:

  • 31 instances of E402 (module level import not at top of file)
  • 19 instances of E302 (expected 2 blank lines, found 0)
  • 12 instances of E302 (expected 2 blank lines, found 1)
  • 12 instances of E266 (too many leading '#' for block comment)
  • 9 instances of E225 (missing whitespace around operator)
  • 9 instances of E127 (continuation line over-indented for visual indent)
  • 9 instances of E226 (missing whitespace around arithmetic operator)
  • 8 instances of E301 (expected 1 blank line, found 0)
  • 6 instances of E305 (expected 2 blank lines after class or function definition, found 0)
  • 3 instances of E231 (missing whitespace after ',')

@spirosChv spirosChv merged commit eef0991 into main Jul 19, 2022
@spirosChv spirosChv deleted the w2d5t2_fixes branch July 19, 2022 19:40
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

Successfully merging this pull request may close these issues.

W2D5 - Broken image links
2 participants