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

Configurator #467

Merged
merged 25 commits into from Nov 11, 2019
Merged

Configurator #467

merged 25 commits into from Nov 11, 2019

Conversation

sellendk
Copy link
Collaborator

@sellendk sellendk commented Nov 7, 2019

lua import and slot name highlighting with right click connection

…it is not yet completely tested. Also some features are missing, such as arranging the modules at the same position as when they were saved. Also exception checks are missing completely, so beware. Still, better than no import
…ues. Also encountered new issues with inconsistent naming in lua project files, fix is in progress
… reads the confPos of the corresponding modules. Adapted the load function to now correctly read and load modules if the name in the lua file does not contain the project name (see testspheres.lua for example)
…would go out of bounds and added a feature: pressing middle mousebutton while hovered over a module now removes the (top) module
…slashes got doubled every time the file was read
@sellendk sellendk requested a review from reinago November 7, 2019 14:35
Copy link
Contributor

@tobiasrau tobiasrau left a comment

Choose a reason for hiding this comment

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

The configurator changes should be separated from the rest of the changes in this pull request.

Comment on lines 401 to 407
<<<<<<< HEAD
gl_FragColor = vec4(LocalLighting(ray, normal, lightPos.xyz, color), 1.0);
=======
normal = rotMatIT * normal;
ray = rotMatIT * ray;
gl_FragColor = vec4(LocalLighting(ray, normal, lightDir.xyz, color), 1.0);
>>>>>>> upstream/master
Copy link
Contributor

Choose a reason for hiding this comment

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

There is definitely something wrong...

@tobiasrau tobiasrau added missing in changelog List the changes in the wiki, please! enhancement labels Nov 7, 2019
@tobiasrau tobiasrau merged commit 98c15f6 into UniStuttgart-VISUS:master Nov 11, 2019
@tobiasrau tobiasrau removed this from Done in Road to MegaMol 1.4 Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement missing in changelog List the changes in the wiki, please!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants