[Bug] Image animation not working in GTK

[ysangkok]

Description

It seems like image animation is not working in GTK. Similar code worked on the UWP backend.

Steps to Reproduce

https://github.com/ysangkok/XamarinGTKAnimationIssue

src.zip

Expected Behavior

Image should be animated.

Actual Behavior

Image is not animated.

Basic Information

• Version with issue: 4.4.0.991210-pre2
• Last known good version: don't know
• Platform Target Frameworks: Ubuntu 18.04 with gtk-sharp 2.12.45-0xamarin17+ubuntu1804
• Nuget Packages: See linked repository

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

This issue now has a funding of 1.045 ETH (150.03 USD @ $143.57/ETH) attached to it.

• If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
• Want to chip in? Add your own contribution here.
• Questions? Checkout Gitcoin Help or the Gitcoin Slack
• $76,903.23 more funded OSS Work available on the Gitcoin Issue Explorer

[gitcoinbot]

@izoch Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

• reminder (3 days)
• escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

[miloszeka]

Commenting back

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

@izoch due to inactivity, we have escalated this issue to Gitcoin's moderation team. Let us know if you believe this has been done in error!

• reminder (3 days)
• escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

[mul53]

@samhouts does this require work? Would like to work on it

[knocte]

@mul53 what do you mean?

[mul53]

@knocte working on the bug

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

Work has been started.

These users each claimed they can complete the work by 1 week ago.
Please review their action plans below:

1) bobface has been approved to start work.

I can fix the bug with the animation.

Learn more on the Gitcoin Issue Details page.

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

Work for 1.045 ETH (272.64 USD @ $260.9/ETH) has been submitted by:

1. @bobface

@knocte please take a look at the submitted work:

• PR by @Bobface

---

• Learn more on the Gitcoin Issue Details page
• Want to chip in? Add your own contribution here.
• Questions? Checkout Gitcoin Help or the Gitcoin Slack
• $82,169.00 more funded OSS Work available on the Gitcoin Issue Explorer

[ysangkok]

I tested this, with the sample project from the issue. It works, but it consumes 100% CPU and is not smooth. I am not sure whether this is a problem with this PR or with the sample project (which I made).

[Bobface]

Hmm, would it be possible to link the image you have used?

[Bobface]

@ysangkok Also, the large the image the more expensive the calculation for scale & rotate. Did you test if it works smoothly with a small image, say 50x50?

[ysangkok]

@Bobface I have a HiDPI monitor, so 50x50 is really small. But you are right that smaller sizes have less CPU load. For example, a 250x188 image runs with only 50% CPU utilization. I am using the ImageMagick logo image. To generate it in its full size (with width 500px), use convert logo: logo.png.

[knocte]

@Bobface maybe have a look at these answers: https://stackoverflow.com/questions/6888965/any-way-to-speed-up-reduce-cpu-usage-when-drawing-with-cairo?rq=1 :)

[Bobface]

@ysangkok Okay, so the size of the image is definitely the problem.
@knocte Thanks, but it seems like I already do everything as described in that thread.

[knocte]

Even the FPS reduction?

[Bobface]

That should be achievable by reducing the frequency rate of the animation in @ysangkok 's repro (dummyFrequencyRate in LoadingPage.xaml.fs). However he links to another issue which says that that parameter is also not implemented in Xamarin.Forms and has no effect. Is that still correct @ysangkok ?

[ysangkok]

@Bobface I have tested your PR 9856 with this one cherry-picked on top. Everything works as expected. Thank you.

[knocte]

@ysangkok you mean CPU doesn't go to 100% anymore?

[ysangkok]

@knocte CPU consumption is not too high with reasonable animation update intervals and smaller image sizes, no.

[Solexplorer]

Hi @ysangkok,

Is the fix provided by @Bobface working?

[ysangkok]

@Solexplorer yes

[Solexplorer]

@knocte Does this bounty also need review from maintainers?

[knocte]

Yes but this should be merged only after PR 9856 is merged.

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

This issue now has a funding of 1.045 ETH (249.23 USD @ $238.5/ETH) attached to it.

• If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
• Want to chip in? Add your own contribution here.
• Questions? Checkout Gitcoin Help or the Gitcoin Chat
• $307,869.71 more funded OSS Work available on the Gitcoin Issue Explorer

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

Work has been started.

These users each claimed they can complete the work by 1 month from now.
Please review their action plans below:

1) bobface has started work.

.................................................................

Learn more on the Gitcoin Issue Details page.

[gitcoinbot]

⚡️ A tip worth 1.04000 ETH (248.04 USD @ $238.5/ETH) has been granted to @Bobface for this issue from @knocte. ⚡️

Nice work @Bobface! Your tip has automatically been deposited in the ETH address we have on file.

• $307873.51 in Funded OSS Work Available at: https://gitcoin.co/explorer
• Incentivize contributions to your repo: Send a Tip or Fund a PR
• No Email? Get help on the Gitcoin Chat

[knocte]

Fixed by #9649 which was merged , @ysangkok you can close

[gitcoinbot]

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

---

This Bounty has been completed.

Additional Tips for this Bounty:

• knocte tipped 1.0400 ETH worth 248.04 USD to bobface.

---

• Learn more on the Gitcoin Issue Details page
• Questions? Checkout Gitcoin Help or the Gitcoin Chat
• $307,873.51 more funded OSS Work available on the Gitcoin Issue Explorer