René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Invalid cast from BuiltinFunctionInfo to BuiltinClassInfo in GetBuiltinType","articleBody":"From #391:\r\n\r\n```\r\n[Error - 5:37:55 PM] System.InvalidCastException: Unable to cast object of type 'Microsoft.PythonTools.Analysis.Values.BuiltinFunctionInfo' to type 'Microsoft.PythonTools.Analysis.Values.BuiltinClassInfo'.\r\n   at Microsoft.PythonTools.Analysis.PythonAnalyzer.GetBuiltinType(IPythonType type)\r\n   at System.Linq.Enumerable.WhereSelectListIterator`2.MoveNext()\r\n   at System.Collections.Generic.List`1.AddEnumerable(IEnumerable`1 enumerable)\r\n   at System.Collections.Generic.List`1.InsertRange(Int32 index, IEnumerable`1 collection)\r\n   at Microsoft.PythonTools.Analysis.Values.Mro..ctor(IEnumerable`1 values)\r\n   at Microsoft.PythonTools.Analysis.Values.BuiltinClassInfo.get_Mro()\r\n   at Microsoft.PythonTools.Analysis.Values.Mro.Recompute()\r\n   at Microsoft.PythonTools.Analysis.Values.ClassInfo.SetBases(IEnumerable`1 bases)\r\n   at Microsoft.PythonTools.Analysis.ClassAnalysisUnit.AnalyzeWorker(DDG ddg, CancellationToken cancel)\r\n   at Microsoft.PythonTools.Analysis.Analyzer.DDG.Analyze(Deque`1 queue, CancellationToken cancel, Action`1 reportQueueSize, Int32 reportQueueInterval)\r\n   at Microsoft.PythonTools.Analysis.PythonAnalyzer.AnalyzeQueuedEntries(CancellationToken cancel)\r\n   at Microsoft.PythonTools.Intellisense.AnalysisQueue.GroupAnalysis.Analyze(CancellationToken cancel)\r\n   at Microsoft.PythonTools.Intellisense.AnalysisQueue.HandleAnalyzable(IAnalyzable item, AnalysisPriority priority, CancellationToken cancellationToken)\r\n   at Microsoft.PythonTools.Intellisense.AnalysisQueue.ConsumerLoop()\r\n```\r\n\r\n#439 changed a bunch of the MRO stuff, and that PR's timing lines up with when @jarshwah saw this.\r\n\r\nIt seems to me like either:\r\n- We now add more things to the MRO, which the (older) GetBuiltinType method needs to better handle.\r\n- The check in GetBuiltinType is correct, and the new code adds things it shouldn't to the MRO.","author":{"url":"https://github.com/jakebailey","@type":"Person","name":"jakebailey"},"datePublished":"2018-12-08T19:32:23.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/470/python-language-server/issues/470"}