Skip to content

Bring back convert_to_hw_layers imports lost in 1616b42#203

Open
iksnagreb wants to merge 1 commit into
devfrom
fix/lost-imports
Open

Bring back convert_to_hw_layers imports lost in 1616b42#203
iksnagreb wants to merge 1 commit into
devfrom
fix/lost-imports

Conversation

@iksnagreb
Copy link
Copy Markdown

No description provided.

@github-actions
Copy link
Copy Markdown

📋 Docstring Check Report

Docstring check failed!

Missing Docstrings Details:

📄 src/finn/transformation/fpgadataflow/convert_to_hw_layers.py:

  • Line 473: function 'InferRequantLayer.init'
  • Line 476: function 'InferRequantLayer.apply'
  • Line 783: function 'InferAddStreamsLayer.apply'
  • Line 954: function 'InferChannelwiseLinearLayer.apply'
  • Line 1684: function 'InferStreamingEltwise.apply'
  • Line 2192: function 'InferShuffle.init'

Total missing docstrings: 6

How to Fix:

Please add docstrings to the missing functions, classes, and modules listed above.

Docstring Guidelines:

  • All modules should have a module-level docstring
  • All public functions and methods should have docstrings
  • All private functions should have docstrings
  • All classes should have docstrings
  • Use triple quotes (""") for docstrings
  • Follow PEP 257 conventions
Raw output from docstring checker
Checking 1 specified Python file(s) (excluding tests folder):
  - src/finn/transformation/fpgadataflow/convert_to_hw_layers.py

❌ Missing docstrings found:

📄 src/finn/transformation/fpgadataflow/convert_to_hw_layers.py:
  - Line 473: function 'InferRequantLayer.__init__'
  - Line 476: function 'InferRequantLayer.apply'
  - Line 783: function 'InferAddStreamsLayer.apply'
  - Line 954: function 'InferChannelwiseLinearLayer.apply'
  - Line 1684: function 'InferStreamingEltwise.apply'
  - Line 2192: function 'InferShuffle.__init__'

Total missing docstrings: 6

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.

1 participant