CAniImageBox Loading Optimized

[Distraught 10074]

Hello guys,

As we know, if you have a lot of images in an ani-image, it can be really slow to load and may makes the game freeze for a moment. It really limits us, because we can't use a longer image sequence as it would have a bad impact on user experience.

I created a sequence of 126 images by concatenating some of our tutorials, so here's a comparison GIF that shows the old (left) and new (right) version.

The main problem is that it loads all the images in a loop when we initialize it.

And they are loaded when we call AppendImage function.

 

We can delay and separate the loading of images in the update cycle to different frames, so the game won't freeze because just one image per frame will be loaded instead of them all at once.

 

How to do this? Step by step tutorial.

This is the hidden content, please

• Sign In
• or
• Sign Up

 

That's all, we're finished!

Good luck!

Edited July 8, 2022 by Distraught

[Ezequiel G. 532]

Nice, thanks for sharing!

[Kafa 178]

Thank you bro, as always good work!

[Dios 1]

Thanks for ur work.

[meneleos 407]

There is a problem with clearimages function.

	void CAniImageBox::ClearImages()
	{
		while (!m_ImageFileNames.empty())
			m_ImageFileNames.pop();

		for_each(m_ImageVector.begin(), m_ImageVector.end(), CGraphicExpandedImageInstance::DeleteExpandedImageInstance);
		m_ImageVector.clear();
	}

fixed.

 

Edited July 20, 2022 by meneleos

[Gurgarath 3706]

On 7/20/2022 at 2:28 PM, meneleos said:

There is a problem with clearimages function.

	void CAniImageBox::ClearImages()
	{
		while (!m_ImageFileNames.empty())
			m_ImageFileNames.pop();

		for_each(m_ImageVector.begin(), m_ImageVector.end(), CGraphicExpandedImageInstance::DeleteExpandedImageInstance);
		m_ImageVector.clear();
	}

fixed.

 

Are you talking about the destructor? Because there is no "ClearImages" function in clean sources

[meneleos 407]

On 7/25/2022 at 10:29 AM, Gurgarath said:

Are you talking about the destructor? Because there is no "ClearImages" function in clean sources

A necessary function for some systems that were added later.

[Distraught 10074]

7 hours ago, meneleos said:

A necessary function for some systems that were added later.

Then it's not a fix, it's in your code only.

[Ulthar 5298]

On 7/8/2022 at 2:12 AM, Distraught said:

Go to CAniImageBox::SetDiffuseColor function and overwrite with 

	void CAniImageBox::SetDiffuseColor(float fR, float fG, float fB, float fA)
	{
		m_SetDiffuseColor = std::bind(&CGraphicExpandedImageInstance::SetDiffuseColor, std::placeholders::_1, fR, fG, fB, fA);
		std::for_each(m_ImageVector.begin(), m_ImageVector.end(), m_SetDiffuseColor);
	}

Instead of this function,

 have this:

	void CImageBox::SetDiffuseColor(float fr, float fg, float fb, float fa)
	{
		if (!m_pImageInstance)
			return;

		m_pImageInstance->SetDiffuseColor(fr, fg, fb, fa);
	}

Is it okey if i overwrite this?

Edited January 7, 2023 by Ulthar

[Distraught 10074]

That’s the function of another class.

[Ulthar 5298]

29 minutes ago, Distraught said:

That’s the function of another class.

Yea i knew, just asked. so i dont have it, then i do nothing, right ? :'D