Just a general comment. I noticed that the import of this module (and others) is not named consistently.
I have seen actionTypes been imported as Action, t, AT... which makes it somewhat confusing.
Not a major issue, but having a consistent naming of modules could improve readability (especially if you are not very familiar with the codebase :smile: )
Just a general comment. I noticed that the import of this module (and others) is not named consistently. I have seen actionTypes been imported as
Action
,t
,AT
... which makes it somewhat confusing. Not a major issue, but having a consistent naming of modules could improve readability (especially if you are not very familiar with the codebase :smile: )_Originally posted by @kverstae in https://github.com/aertslab/SCope/pull/500#discussion_r817801708_