algunos cambios propuestos #529
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Entiendo los cambios propuestos, no es necesario explicar toda la deducción en este caso y da mas claridad a la explicación. En este caso luego de aprobar el review debería dar un Merge al PR correcto? |
Contributor
|
Correcto @jorgepz la solución la valide de un caso ejemplo de un libro junto a otros ejemplos llegando al mismo resultados. |
Member
Author
|
luego de que vos apruebes el review, yo hago el merge. |
santiago-correa-89
approved these changes
Jul 26, 2022
Contributor
santiago-correa-89
left a comment
There was a problem hiding this comment.
Understood all the changes and the idea. I will add the missing info and double check the result.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
propongo algunos cambios.
el mas grande es sacar las figuras... lamentablemente no hemos resuelto aún porqué las docs no muestran bien las flechas... es por eso que las figuras no se ven bien https://onsas.github.io/ONSAS.m/previews/PR526/examples/beamLinearVibration/ lamento no haber avisado antes! es lo mismo que se ve en la primer figura del ejemplo ... una forma de resolverlo es no poner las figuras... no parece ser necesario mostrar toda la deducción de un equilibrio de vigas cuando lo más complicado en realidad está en el salto de tener la ecuación para una carga distribuida y pasar a tener la solución para una fuerza puntual!
otro cambio importante que sugiero es indicar el libro de donde se sacó la solución, esto es así no? la solución se sacó de un libro?
gracias de nuevo por el trabajo @santiago-correa-89 !