Skip to content

feat: smoke simulator #135

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.

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

Merged
merged 13 commits into from
Jun 21, 2023
Merged

feat: smoke simulator #135

merged 13 commits into from
Jun 21, 2023

Conversation

HayatoTier4
Copy link
Collaborator

I am creating a smoke simulator such that the RGL LiDAR will be able to detect the particles of the smoke.
To the best of my knowledge, this cannot be done with Particle System, so there is a need to create a new feature for achieving the task.

The feature is still under development and needs further adjustment.
image

@HayatoTier4 HayatoTier4 changed the title Feat/smoke simulator feat:smoke simulator Jun 5, 2023
@HayatoTier4 HayatoTier4 added the enhancement New feature or request label Jun 5, 2023
@HayatoTier4 HayatoTier4 changed the title feat:smoke simulator feat: smoke simulator Jun 5, 2023
@HayatoTier4 HayatoTier4 marked this pull request as ready for review June 5, 2023 09:14
@HayatoTier4
Copy link
Collaborator Author

Here is an image of the smoke simulator compared with particle system. The particles of the smoke simulator is detected by the LiDAR.
The parameters can be adjusted to increase/decrease the number of particles, and adjust the shape and behavior of the smoke.

image_0

Copy link
Collaborator

@mackierx111 mackierx111 left a comment

Choose a reason for hiding this comment

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

Thank you for the implementation. Some feedback.

  • Please move the SmokeGenerator script directory under Environments.
    • Assets\AWSIM\Scripts\Environments\SmokeGenerator
  • Please create a SmokeGenerator prefab.
  • Could you add a comment for the public attribute member or class of each script? (Not necessary for those that have comments written in Tooltip)

@HayatoTier4 HayatoTier4 requested a review from mackierx111 June 19, 2023 01:43
@mackierx111 mackierx111 marked this pull request as draft June 19, 2023 07:55
@mackierx111 mackierx111 marked this pull request as ready for review June 21, 2023 05:51
Copy link
Collaborator

@mackierx111 mackierx111 left a comment

Choose a reason for hiding this comment

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

LGTM👍

@mackierx111 mackierx111 merged commit e896866 into main Jun 21, 2023
@mackierx111 mackierx111 deleted the feat/smoke_simulator branch June 21, 2023 05:52
tyongingo pushed a commit to tyongingo/AWSIM that referenced this pull request Nov 22, 2024
Signed-off-by: A. Sena Yılmaz <aysena@leodrive.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants