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

Feature/maroon 404 falling ball viscosimeter #511

Open
wants to merge 64 commits into
base: develop
Choose a base branch
from

Conversation

danielliegl
Copy link
Collaborator

Close #404

- 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
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
@danielliegl danielliegl marked this pull request as ready for review August 27, 2024 12:13
Copy link
Collaborator

@FlorianGlawogger FlorianGlawogger left a 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! 👍

Copy link
Collaborator

Choose a reason for hiding this comment

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

grafik
If VR is not supported for the experiment, please delete the scene

Copy link
Collaborator

Choose a reason for hiding this comment

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

Incorrect text in laboratory
grafik
(missing translation in Assets/Resources/MLG/laboratory.xml probably)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Colliders are too large / too many colliders
grafik
(only the table and info stand sign thing should have colliders)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please delete unnecessary Coulombs Law stuff from this experiment
grafik

//calculate dynamic viscosity
decimal dynamic_viscosity = ViscosimeterManager.Instance.calculateDynamicViscosity(temperature);

Debug.Log("dynamic visc " + dynamic_viscosity);
Copy link
Collaborator

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

When using any fluid other than 1,2-Ethanediol, (e.g. Cyclohexane), then the ball jumps up and down at the bottom of the cylinder tube
cursor

Copy link
Collaborator

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

When placing the ball, I often placed it to the Faucet snap point by accident
grafik
Then the ball was outside the tube, but it still behaved (falling speed) the same as inside the tube filled with fluid
if it's not too much work it would be cool to prevent the ball from snapping to this snap point

Copy link
Collaborator

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)

Copy link
Collaborator

Choose a reason for hiding this comment

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

When clicking Measure Length and then on e.g. the tube, I get the following error
grafik

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.

Vertical Falling Ball Viscometer Experiment
2 participants