-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature/maroon 404 falling ball viscosimeter #511
base: develop
Are you sure you want to change the base?
Feature/maroon 404 falling ball viscosimeter #511
Conversation
not working :(
…//github.com/GameLabGraz/Maroon into feature/Maroon-404_Falling_Ball_Viscosimeter
- Ball und Röhre bewegt, damit es besser in schirm passt - Pyknometer leergewicht bug behoben - Hahn für Pyknometer - Messmodus verbessert - UI Elemente auf die notwendigen reduziert - Ball Startgröße vergrößert
…rag when measuring.
…//github.com/GameLabGraz/Maroon into feature/Maroon-404_Falling_Ball_Viscosimeter
Moved Fluid property calculation to ViscosimeterManager
- Make Reset Ball more clear - Make it more clear that slow motion is active. - Hint for zooming during measurements. - Change key binding to mouse2 for panning in zoom mode - disable collider of measured object - Do not reset temperature after resetting the simulation. Added bool for allowing reset in the QuantityPropertyView
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The experiment is really cool, and very well done! I think it is really well-suited to conduct experiments with it and actually learn about viscosity. 😄
I made a lot of remarks but mostly about code formatting, naming of methods, missing namespaces etc., so the vast majority of comments will not take up lots of time.
Please also merge with develop branch.
But again: very cool experiment! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
//calculate dynamic viscosity | ||
decimal dynamic_viscosity = ViscosimeterManager.Instance.calculateDynamicViscosity(temperature); | ||
|
||
Debug.Log("dynamic visc " + dynamic_viscosity); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove this Debug.Log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not too big of a deal i suppose, but when the ball rotates, the ball camera rotates as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it on purpose that the tooltip for the tube only appears above and below the red lines? (not in the center)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Close #404