framelist of sprite is wrong when using GUIX studio 6.1.0.0 about guix

[jltrahan-2020]

I second this bug... It's been around since at least v5.5 or 5.6 of GUIX Studio (back before the MS acquisition.) As Wesley reported above, GUIX Studio generates the wrong indices in the framelist. The GX_SPRITE_FRAME array of frames have the wrong pixelmap id in the specifications.c file. This shows in both the GUIX Studio interface, and the target compiled code.

The obvious workaround is to manually edit this .c file, but that's a pain in the long run, as it's not intended to be used this way.

EDIT: was referring to .h files, but it's the .c files that are incorrect.

[jdeere5220]

Can you tell me what version of GUIX Studio you are using? I will look into this. We have several sprite based regression tests in place that are not exhibiting this issue so we will have to figure out what causes this to happen. The next GUIX Studio update is set for the end of this month, and we will do our best to duplicate this issue and resolve it for the next update. GUIX Studio is moving to the Microsoft App Store for the next release.

[jltrahan-2020]

As the subject of the issue indicates, this is on the latest version 6.1.0.0. I will send a private email with an example GUIX project.

[jltrahan-2020]

@jdeere5220 Can you confirm that you received a private mail with a sample project? I've even created another one from scratch using 6.1.0.0, thinking that importing an older .gxp file (generated by GUIX 5.5) would cause the issue, but even a fresh new project does not generate the correct pixelmap id's in the frame array.

[jdeere5220]

@jltrahan-2020 Sorry I did not notice the thread title contained the version information. I did not receive a private email but I am looking into the issue. We will try to have it fixed for our end-of-this-month release.

[jltrahan-2020]

Let me know if you need someone to test the correction. I'll take a look here once in a while during the Holidays.

Thanks!

[jdeere5220]

@jltrahan-2020 we have reproduced the issue and fixed it (and added a new test to be sure it never comes back). The fix will be in our next Studio release, which I believe will be 6.1.3.0, early January.

[jdeere5220]

@jltrahan-2020 just a side note, we did have a regression test in place for this code generation, but it turns out the test was in this case completely useless. The test project included only pixelmaps used in the sprite, and they were defined in alphabetical order, and used in alphabetical order, so of course the generated output was, in this case, correct. We have greatly improved the test going forward!

[jltrahan-2020]

Thanks Ken, we look forward to using the update version!

[yuxin-azrtos]

Hi @jltrahan-2020 we have released an updated GUIX Studio you can find at https://aka.ms/azrtos-guix-installer

Let us know if the newer version fixes the problem. Thanks!

[jltrahan-2020]

Yes, the issue is fixed. From: yuxin-azrtos <[email protected]> Sent: January 29, 2021 11:27 To: azure-rtos/guix <[email protected]> Cc: Jean-Luc Trahan <[email protected]>; Mention <[email protected]> Subject: Re: [azure-rtos/guix] framelist of sprite is wrong when using GUIX studio 6.1.0.0 (#11) Hi @jltrahan-2020<https://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjltrahan-2020&data=04%7C01%7C%7C12d73ea15d4343fe6f8c08d8c472a822%7Cf193f6f562be492d897d776cde125585%7C1%7C0%7C637475344026123893%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=qVcY7hPGSSpWJtCOmNMoBqS4h26yPqvttnwurYd7sig%3D&reserved=0> we have released an updated GUIX Studio you can find at https://aka.ms/azrtos-guix-installer<https://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2Fazrtos-guix-installer&data=04%7C01%7C%7C12d73ea15d4343fe6f8c08d8c472a822%7Cf193f6f562be492d897d776cde125585%7C1%7C0%7C637475344026123893%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=10R2mXJgS3rRG25ZAqnhduqIfTSc9XyXAi8u%2FBtaBs8%3D&reserved=0> Let us know if the newer version fixes the problem. Thanks! - You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fazure-rtos%2Fguix%2Fissues%2F11%23issuecomment-769906569&data=04%7C01%7C%7C12d73ea15d4343fe6f8c08d8c472a822%7Cf193f6f562be492d897d776cde125585%7C1%7C0%7C637475344026133889%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=UOLIgrNFqiPrQBz2BKbBc7dN0GugSHHlUm6aVULNsmY%3D&reserved=0>, or unsubscribe<https://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASHQ677DX55LSB4SKQHC2ITS4LOT5ANCNFSM4U3U6V6Q&data=04%7C01%7C%7C12d73ea15d4343fe6f8c08d8c472a822%7Cf193f6f562be492d897d776cde125585%7C1%7C0%7C637475344026133889%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=PzMqRr7NCV9r7Q2LFDq66xsm9aX%2FHZo0gg4VvSeLyT0%3D&reserved=0>.

[yuxin-azrtos]

Thanks!