[UWP][Memory] [Rendered Cards Leaking Memory]

Platform

• UWP

Version of SDK

AdaptiveCards.Redering.Uwp V1.2.2

Details

When redering an adaptive card, it appears that its memory is not released. The bellow code sample loads an adaptive card, displays it on the UI before removing it and forcing a garbage collection. After 50 itteraltions, an additional 90MB of memory is being used by the test app.

var renderer = new AdaptiveCardRenderer();
string card = "{   \"$schema\": \"http://adaptivecards.io/schemas/adaptive-card.json\",   \"type\": \"AdaptiveCard\",   \"version\": \"1.0\",   \"body\": [     {       \"type\": \"TextBlock\",       \"text\": \"Publish Adaptive Card schema\",       \"weight\": \"bolder\",       \"size\": \"medium\"     },     {       \"type\": \"ColumnSet\",       \"columns\": [         {           \"type\": \"Column\",           \"width\": \"auto\",           \"items\": [             {               \"type\": \"Image\",               \"url\": \"https://pbs.twimg.com/profile_images/3647943215/d7f12830b3c17a5a9e4afcc370e3a37e_400x400.jpeg\",               \"size\": \"small\",               \"style\": \"person\"             }           ]         },         {           \"type\": \"Column\",           \"width\": \"stretch\",           \"items\": [             {               \"type\": \"TextBlock\",               \"text\": \"Matt Hidinger\",               \"weight\": \"bolder\",               \"wrap\": true             },             {               \"type\": \"TextBlock\",               \"spacing\": \"none\",               \"text\": \"Created {{DATE(2017-02-14T06:08:39Z, SHORT)}}\",               \"isSubtle\": true,               \"wrap\": true             }           ]         }       ]     },     {       \"type\": \"TextBlock\",       \"text\": \"Now that we have defined the main rules and features of the format, we need to produce a schema and publish it to GitHub. The schema will be the starting point of our reference documentation.\",       \"wrap\": true     },     {       \"type\": \"FactSet\",       \"facts\": [         {           \"title\": \"Board:\",           \"value\": \"Adaptive Card\"         },         {           \"title\": \"List:\",           \"value\": \"Backlog\"         },         {           \"title\": \"Assigned to:\",           \"value\": \"Matt Hidinger\"         },         {           \"title\": \"Due date:\",           \"value\": \"Not set\"         }       ]     }   ],   \"actions\": [     {       \"type\": \"Action.ShowCard\",       \"title\": \"Set due date\",       \"card\": {         \"type\": \"AdaptiveCard\",         \"body\": [           {             \"type\": \"Input.Date\",             \"id\": \"dueDate\"           }         ],         \"actions\": [           {             \"type\": \"Action.Submit\",             \"title\": \"OK\"           }         ]       }     },     {       \"type\": \"Action.ShowCard\",       \"title\": \"Comment\",       \"card\": {         \"type\": \"AdaptiveCard\",         \"body\": [           {             \"type\": \"Input.Text\",             \"id\": \"comment\",             \"isMultiline\": true,             \"placeholder\": \"Enter your comment\"           }         ],         \"actions\": [           {             \"type\": \"Action.Submit\",             \"title\": \"OK\"           }         ]       }     }   ] }";

for (var i = 0; i < 50; ++i)
{
    await Task.Delay(1);
    this.ContentControl.Content = null;

    var card = AdaptiveCard.FromJsonString(this.card);

    if (card != null)
    {
        var renderedAdaptiveCard = renderer.RenderAdaptiveCard(card.AdaptiveCard);

        if (renderedAdaptiveCard != null)
        {
            this.ContentControl.Content = renderedAdaptiveCard.FrameworkElement;
        }
    }
    else
    {
        await (new MessageDialog("Card was null", ":(")).ShowAsync();
    }

    await Task.Delay(100);
    GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
}

[barryptak]

Hey, @paulcam206. Are you able to be clear on why this P1 bug is this not on the roadmap? Is it because it’s been mitigated significantly and so is not considered a major issue anymore? Is it because UWP is not being supported going forward? Is it something else?

Thanks!

[paulcam206]

Addressing this issue is not currently on our roadmap, so closing. Please feel free to continue the conversation in this issue for future consideration.