Variable annotation triggers exception 'Subscript' object has no attribute 'id' #93

Activity

1. gjdm

   changed the title [-]Variable annotation triggers exception 'Suscript' object has no attribute 'id'[/-] [+]Variable annotation triggers exception 'Subscript' object has no attribute 'id'[/+] on Jun 7, 2021
2. abulka commented on Jun 7, 2021

   

   abulka

   on Jun 7, 2021

   Owner

   More actions

   Thank you for reporting this.
3. abulka commented on Jun 29, 2021

   

   abulka

   on Jun 29, 2021

   Last edited by abulka

   Owner

   More actions

   Tried to reproduce this running under Python 3.9 parsing the following:

   class A:
       def method1(self):
           self.pages: List[SnapFileView2D] = list()

   and got no errors.

   Hmm - perhaps this error only happens with lower version of Python - will need to check. As it stands, it looks promising that no changes to the Pynsource parser are needed, only a Python version bump. That is, assuming the above source code reproduces the problem.

   Update: Tricky variable annotation, Actually turns out that on my mac, it used to fail ONLY in Python 3.9 but passed in 3.7 and 3.8.
4. abulka

   added a commit that references this issue on Jun 29, 2021

   test created for issue #93

   2b9d09a
5. abulka commented on Jun 30, 2021

   

   abulka

   on Jun 30, 2021

   Owner

   More actions

   Tested parsing this on Mac using official release which uses Python 3.7 internally - worked ok.

   But on Linux the official snap release does exhibit the problem, but running under Python 3.7 from source code does not. Mysterious. Will investigate further.
6. abulka

   added

   Component-Parsing

   Priority-Medium

   and removed

   Priority-Medium

   on Jul 1, 2021
7. abulka

   added a commit that references this issue on Jul 1, 2021

   better tests and parsing for issue #85, #93, #94 - tests pass under p…

   ...

   ff83263
8. abulka commented on Jul 1, 2021

   

   abulka

   on Jul 1, 2021

   Owner

   More actions

   Changed visit_AnnAssign() in parser to fix this, passes now under all Python versions.
9. abulka commented on Jul 8, 2021

   

   abulka

   on Jul 8, 2021

   Owner

   More actions

   Latest master branch and 1.78 beta binary release should have fixed this.
   Official 1.78 release only a few days away.

   👍React with 👍1Reacted by Peter
10. abulka

    closed this as completedon Jul 8, 2021