Skip to content

Fix malformed 'pytlint' lint pragma -> 'pylint' in hd configs - #756

Open
PratikDhanave wants to merge 1 commit into
google-deepmind:mainfrom
PratikDhanaveFork:fix/diffusion-pytlint-pragma-typo
Open

Fix malformed 'pytlint' lint pragma -> 'pylint' in hd configs#756
PratikDhanave wants to merge 1 commit into
google-deepmind:mainfrom
PratikDhanaveFork:fix/diffusion-pytlint-pragma-typo

Conversation

@PratikDhanave

Copy link
Copy Markdown

Three hackable-diffusion config files use # pytlint: disable=unused-import to silence the intentional _paths import. pytlint is not a recognized pragma prefix, so the warning is never actually suppressed. Correct it to pylint in:

  • hackable_diffusion_adapter/configs/sft_sudoku.py
  • hackable_diffusion_adapter/configs/sft_pubmedqa.py
  • hackable_diffusion_adapter/configs/sft_sudoku_full.py

'# pytlint: disable=unused-import' is not a recognized pragma, so the
unused-import warning it was meant to silence is never suppressed. Fix the
directive to 'pylint' in the three sft_* config files.
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