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

fix direction of lights on robot robotika/Freyja #588

Merged
merged 1 commit into from
Aug 31, 2020

Conversation

m3d
Copy link
Contributor

@m3d m3d commented Aug 30, 2020

The lights unfortunately pointed to the ground - fixed now.
thanks
Martin/Robotika Team

@nkoenig
Copy link
Contributor

nkoenig commented Aug 31, 2020

Lighting before:
freyja_before

Lighting after:
freyja_after

Looks good. This will make it into the code freeze.

@nkoenig nkoenig merged commit ee0c90b into osrf:master Aug 31, 2020
@m3d
Copy link
Contributor Author

m3d commented Aug 31, 2020

thanks a lot :)
m.

@m3d
Copy link
Contributor Author

m3d commented Sep 10, 2020

Is there a way how to check that this was really deployed on CloudSim? We would like to fix also configuration 2 (after "unfreeze") and they both look the same??
git-freyja1
Freyja config1
git-with-breadcrums
Freyja config2

@nkoenig
Copy link
Contributor

nkoenig commented Sep 11, 2020

Those lighting issues are due to the javascript library used to render the scene. A fix has been applied here and needs to be rolled out to the portal. The docker images do not require changes.

@m3d
Copy link
Contributor Author

m3d commented Sep 13, 2020

@nkoenig great - thanks for info

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