Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix setting particle size #241

Merged
merged 7 commits into from
Feb 17, 2021
Merged

Fix setting particle size #241

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
a90cb18
fix particle size
iche033 Feb 10, 2021
d58deef
remove comments
iche033 Feb 10, 2021
4afde4e
update example
iche033 Feb 10, 2021
7b38ed1
update fix
iche033 Feb 12, 2021
cd86e73
Merge branch 'ign-rendering4' into particle_size
iche033 Feb 12, 2021
a19125e
review feedback
adlarkin Feb 12, 2021
a84dbbf
Merge branch 'ign-rendering4' into particle_size
adlarkin Feb 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions examples/particles_demo/Main.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,7 @@ void buildScene(ScenePtr _scene)

//! [create particle emitter]
ParticleEmitterPtr emitter = _scene->CreateParticleEmitter();
emitter->SetType(EM_POINT);
emitter->SetLocalPose({2, 1.10, 1.25, 1.5708, 0, 2.3});
emitter->SetRate(10);
emitter->SetParticleSize({1, 1, 1});
Expand All @@ -118,6 +119,21 @@ void buildScene(ScenePtr _scene)
emitter->SetEmitting(true);
root->AddChild(emitter);
//! [create particle emitter]

// area emitter
ParticleEmitterPtr areaEmitter = _scene->CreateParticleEmitter();
areaEmitter->SetType(EM_BOX);
areaEmitter->SetEmitterSize({3.0, 3.0, 3.0});
areaEmitter->SetLocalPose({3, 0, 0, 0, -1.5707, 0});
areaEmitter->SetRate(10);
areaEmitter->SetParticleSize({0.01, 0.01, 0.01});
areaEmitter->SetLifetime(1);
areaEmitter->SetVelocityRange(0.5, 1);
areaEmitter->SetMaterial(particleMaterial);
areaEmitter->SetColorRangeImage(RESOURCE_PATH + "/smokecolors.png");
areaEmitter->SetScaleRate(1);
areaEmitter->SetEmitting(true);
root->AddChild(areaEmitter);
Comment on lines +123 to +136
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the tutorial need to be updated at all to make reference to this new area emitter?

}

//////////////////////////////////////////////////
Expand Down
7 changes: 7 additions & 0 deletions ogre2/include/ignition/rendering/ogre2/Ogre2ParticleEmitter.hh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,13 @@ namespace ignition
// Documentation inherited.
protected: virtual void Init() override;

/// \brief Internal pre-render function added to avoid breaking ABI
/// compatibility
private: void PreRenderImpl();

/// \brief Create the particle system
private: void CreateParticleSystem();

/// \brief Only the ogre scene can instanstiate this class
private: friend class Ogre2Scene;

Expand Down
123 changes: 88 additions & 35 deletions ogre2/src/Ogre2ParticleEmitter.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,7 @@ class ignition::rendering::Ogre2ParticleEmitterPrivate
public: Ogre::ParticleSystem *ps = nullptr;

/// \brief Ogre particle emitter.
public: std::array<Ogre::ParticleEmitter*,
EmitterType::EM_NUM_EMITTERS> emitters;
public: Ogre::ParticleEmitter *emitter = nullptr;

// \brief Ogre color image affector.
public: Ogre::ParticleAffector *colorImageAffector = nullptr;
Expand All @@ -62,6 +61,10 @@ class ignition::rendering::Ogre2ParticleEmitterPrivate

/// \brief Pointer to the unlit material used by particle emitter.
public: MaterialPtr materialUnlit;

/// \brief Flag to indicate that the emitter is dirty and needs to be
/// recreated
public: bool emitterDirty = false;
};

// Names used in Ogre for the supported emitters.
Expand Down Expand Up @@ -91,6 +94,11 @@ void Ogre2ParticleEmitter::Destroy()
{
if (this->dataPtr->ps)
{
this->dataPtr->ps->removeAllAffectors();
this->dataPtr->colorInterpolatorAffector = nullptr;
this->dataPtr->colorImageAffector = nullptr;
this->dataPtr->scalerAffector = nullptr;

this->scene->OgreSceneManager()->destroyParticleSystem(
this->dataPtr->ps);
this->dataPtr->ps = nullptr;
iche033 marked this conversation as resolved.
Show resolved Hide resolved
Expand All @@ -115,11 +123,16 @@ void Ogre2ParticleEmitter::Ogre2ParticleEmitter::SetType(
return;
}

adlarkin marked this conversation as resolved.
Show resolved Hide resolved
for (auto i = 0; i < EmitterType::EM_NUM_EMITTERS; ++i)
this->dataPtr->emitters[i]->setEnabled(false);
if (this->type == _type)
return;

this->dataPtr->emitters[_type]->setEnabled(true);
this->type = _type;

this->dataPtr->emitterDirty = true;
// todo(anyone) remove this call. We had to do this since we can't override
// PreRender() as it breaks ABI. We should rename PreRenderImp to PreRender()
adlarkin marked this conversation as resolved.
Show resolved Hide resolved
// in next release.
this->PreRenderImpl();
}

//////////////////////////////////////////////////
Expand Down Expand Up @@ -158,15 +171,13 @@ void Ogre2ParticleEmitter::SetEmitterSize(const ignition::math::Vector3d &_size)
for (auto[param, value] : allParamsToSet)
{
// We skip EM_POINT.
for (auto i = 1; i < EmitterType::EM_NUM_EMITTERS; ++i)
if (!this->dataPtr->emitter->setParameter(param, value))
{
if (!this->dataPtr->emitters[i]->setParameter(param, value))
{
ignerr << "SetEmitterSize() error for " << kOgreEmitterTypes[i]
<< " emitter because SetParameter(" << param << " " << value
<< ") failed." << std::endl;
return;
}
ignerr << "SetEmitterSize() error for "
<< this->dataPtr->emitter->getType()
<< " emitter because SetParameter(" << param << " " << value
<< ") failed." << std::endl;
return;
}
}
break;
Expand All @@ -193,25 +204,24 @@ void Ogre2ParticleEmitter::SetRate(double _rate)
return;
}

for (auto i = 0; i < EmitterType::EM_NUM_EMITTERS; ++i)
this->dataPtr->emitters[i]->setEmissionRate(_rate);
this->dataPtr->emitter->setEmissionRate(_rate);

this->rate = _rate;
}

//////////////////////////////////////////////////
void Ogre2ParticleEmitter::SetDuration(double _duration)
{
for (auto i = 0; i < EmitterType::EM_NUM_EMITTERS; ++i)
this->dataPtr->emitters[i]->setDuration(_duration);
this->dataPtr->emitter->setDuration(_duration);


this->duration = _duration;
}

//////////////////////////////////////////////////
void Ogre2ParticleEmitter::SetEmitting(bool _enable)
{
this->dataPtr->emitters[this->type]->setEnabled(_enable);
this->dataPtr->emitter->setEnabled(_enable);
this->dataPtr->ps->setEmitting(_enable);
this->emitting = _enable;
}
Expand All @@ -227,8 +237,14 @@ void Ogre2ParticleEmitter::SetParticleSize(
<< "Particle size values should be non-negative." << std::endl;
return;
}
this->dataPtr->ps->setDefaultDimensions(_size[0], _size[1]);

this->particleSize = _size;

this->dataPtr->emitterDirty = true;
// todo(anyone) remove this call. We had to do this since we can't override
// PreRender() as it breaks ABI. We should rename PreRenderImp to PreRender()
adlarkin marked this conversation as resolved.
Show resolved Hide resolved
// in next release.
this->PreRenderImpl();
}

//////////////////////////////////////////////////
Expand All @@ -242,8 +258,7 @@ void Ogre2ParticleEmitter::SetLifetime(double _lifetime)
return;
}

for (auto i = 0; i < EmitterType::EM_NUM_EMITTERS; ++i)
this->dataPtr->emitters[i]->setTimeToLive(_lifetime);
this->dataPtr->emitter->setTimeToLive(_lifetime);

this->lifetime = _lifetime;
}
Expand All @@ -267,8 +282,8 @@ void Ogre2ParticleEmitter::SetMaterial(const MaterialPtr &_material)
void Ogre2ParticleEmitter::SetVelocityRange(double _minVelocity,
double _maxVelocity)
{
for (auto i = 0; i < EmitterType::EM_NUM_EMITTERS; ++i)
this->dataPtr->emitters[i]->setParticleVelocity(_minVelocity, _maxVelocity);
this->dataPtr->emitter->setParticleVelocity(_minVelocity, _maxVelocity);


this->minVelocity = _minVelocity;
this->maxVelocity = _maxVelocity;
Expand Down Expand Up @@ -413,7 +428,50 @@ void Ogre2ParticleEmitter::SetColorRangeImage(const std::string &_image)
void Ogre2ParticleEmitter::Init()
{
Ogre2Visual::Init();
this->CreateParticleSystem();
}

//////////////////////////////////////////////////
void Ogre2ParticleEmitter::PreRenderImpl()
{
// \todo(anyone) rename this function to PreRender() so it overrides function
// from base class
adlarkin marked this conversation as resolved.
Show resolved Hide resolved

// recreate the particle system if needed
// currently this is needed when user changes type or particle size
if (this->dataPtr->emitterDirty)
{
this->Destroy();
this->CreateParticleSystem();

// make direct ogre calls here so we don't mark emitter as dirty again
this->dataPtr->ps->setDefaultDimensions(
this->particleSize[0], this->particleSize[1]);

this->SetEmitterSize(this->emitterSize);

// set other properties
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe we need to call SetDuration and SetEmitting here as well, since these methods set properties for this->dataPtr->emitter.

Copy link
Contributor

@adlarkin adlarkin Feb 12, 2021 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in a19125e. @iche033 if this looks good to you, feel free to mark this as resolved.

this->SetLifetime(this->lifetime);
this->SetRate(this->rate);
this->SetVelocityRange(this->minVelocity, this->maxVelocity);

if (this->material)
this->SetMaterial(this->material);

if (!this->colorRangeImage.empty())
this->SetColorRangeImage(this->colorRangeImage);
else
this->SetColorRange(this->colorStart, this->colorEnd);

this->SetScaleRate(this->scaleRate);

this->dataPtr->emitterDirty = false;
}
}

//////////////////////////////////////////////////
void Ogre2ParticleEmitter::CreateParticleSystem()
{
// Instantiate the particle system and default parameters.
this->dataPtr->ps = this->scene->OgreSceneManager()->createParticleSystem();
this->dataPtr->ps->setCullIndividually(true);
Expand All @@ -424,16 +482,12 @@ void Ogre2ParticleEmitter::Init()
"The nummer of supported emitters does not match the number of "
"Ogre emitter types.");

// Instantiate all particle emitters and their default parameters.
// Note that we enable the point emitter by default.
for (auto i = 0; i < EmitterType::EM_NUM_EMITTERS; ++i)
{
this->dataPtr->emitters[i] =
this->dataPtr->ps->addEmitter(kOgreEmitterTypes[i]);
this->dataPtr->emitters[i]->setEnabled(false);
this->dataPtr->emitters[i]->setDirection(Ogre::Vector3::UNIT_X);
}
this->dataPtr->emitters[EmitterType::EM_POINT]->setEnabled(true);
// Instantiate particle emitter and their default parameters.
// default in point emitter
adlarkin marked this conversation as resolved.
Show resolved Hide resolved
this->dataPtr->emitter =
this->dataPtr->ps->addEmitter(kOgreEmitterTypes[this->type]);
this->dataPtr->emitter->setDirection(Ogre::Vector3::UNIT_X);
this->dataPtr->emitter->setEnabled(true);

// Instantiate the default material.
this->dataPtr->materialUnlit = this->scene->CreateMaterial();
Expand All @@ -443,8 +497,7 @@ void Ogre2ParticleEmitter::Init()
this->dataPtr->ps->setMaterialName(
*(this->dataPtr->ogreDatablock->getNameStr()));

// Default emitter parameters.
this->SetParticleSize({1, 1, 1});
this->dataPtr->ps->setDefaultDimensions(1, 1);

this->ogreNode->attachObject(this->dataPtr->ps);
igndbg << "Particle emitter initialized" << std::endl;
Expand Down