returnn icon indicating copy to clipboard operation
returnn copied to clipboard

CI: Update PyCharm

Open albertz opened this issue 5 months ago • 0 comments

Our CI GitHub Actions pycharm-inspect job runs tests/pycharm-inspect.py which uses pycharm-community-2020.2.

I usually develop with the latest PyCharm, i.e. currently 2025.1.1.

The differences in code style warnings has so far mostly not been a problem. But now there seem to be such a case: commit c14cd5992e33c707a876880fcfb8e3254935ac81, CI log: returnn/frontend/decoder/transformer.py:364: WARNING PyUnresolvedReferencesInspection: Unresolved attribute reference 'transform_encoder' for class 'Module'

(Btw, maybe also a reminder for #1157. But I don't really think there is any update to that...)

In PR #1743, I was trying to update PyCharm, but in the end reverted back to keeping the 2020.2 version for now, so the PR only contains some other related updates. I think this needs some more work.

I think most crucially, it now finds a couple of valid warnings that were not found before. Those should be fixed.

But then, there are also some more new false positives. We should see what to do about them, if we can easily identify them (but at the same time not ignore huge subsets of potential valid warnings). Some rare warn ignores in the code could also be ok, but this should be done very rarely, and only for well understood cases. Better to discuss this here (or in a new draft PR) before this is done too much.

albertz avatar Jul 24 '25 13:07 albertz