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.

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

Add Configuration Parameters for Robustness Experiments #380

Open
wants to merge 3 commits into
base: noetic-devel
Choose a base branch
from

Conversation

SakshayMahna
Copy link
Contributor

In reference to issue #313

For Camera Position, a temporary model f1_temp is created in the Gazebo models directory. The tmp_world created by tmp_world_generator.py uses this f1_temp model. The example config file is default-multiple-robustness.yml.

For Noise, a new utility script added in utils folder. The noise is added to the camera image just before it is fed to the Neural Network. Example script is brain_f1_explicit_noise.py and example configuration file is default-noise.yml.

@sergiopaniego
Copy link
Member

Nice one!
I have a question regarding the noise addition. Would it be possible adding it as another image transformation option? Following the current pipeline:

image = self.handler.transform_image(image, self.config['ImageTranform'])

What do you think about it?

Is it possible modifying the camera on execution with the graphical interface? You have only provided the multiple config file so I guess that it's only intended for scripting purposes.

@SakshayMahna
Copy link
Contributor Author

Yes, we can add the noise to the current pipeline. The code would be much more organized in that way. In the albumentations library, there is one transformation for Gaussian Noise, but no such function for Salt and Pepper Noise. I can use my custom implementation for the same.

I'm not sure how to modify the camera on execution. But, we can see the noisy image in the graphical execution. How should I go about it? We can possibly do the same for all the transformations present in the handler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants