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

Need to redefine the equation of Impulse #1146

Open
oluowoj opened this issue Feb 28, 2019 · 9 comments
Open

Need to redefine the equation of Impulse #1146

oluowoj opened this issue Feb 28, 2019 · 9 comments
Assignees
Labels
needs-action-items A clear 'path to resolution' is needed to close any ticket. needs-clarification Needs a clear 'state', 'goal', 'analysis', and 'explanation' to reduce solution ambiguities.

Comments

@oluowoj
Copy link
Collaborator

oluowoj commented Feb 28, 2019

Equation of impulse in manual version is shown below:
image

@JacquesCarette will discuss how to define the equation without 2 =

@smiths
Copy link
Collaborator

smiths commented Feb 28, 2019

I don't know if it addresses the concerns of @JacquesCarette, but I think what we want to do is write the final form of the equation that we need and then use the derivation section to show the progression from the general equation to the specific equation. In looking at the impulse equation now, there are definitely problems with it. J is shown bold, as a vector, by m*Delta v is a scalar. How did that happen? There are steps missing. My guess is that we want the scalar J = m * Delta v and we can show a way to derive this from the more general integral form of the impulse equation.

We are also making a transition from the continuous form of the equation to a discrete form.

We'll need to make sure that we have assumptions in place that let us get from the general form to the specific.

@JacquesCarette
Copy link
Owner

I agree that the equation should be written in its final form. And, for now, the derivation can show the rest. We'll want to 'upgrade' that later.

@oluowoj
Copy link
Collaborator Author

oluowoj commented Mar 5, 2019

Impulse(vector) equation has been resolved and updated to
J = m*Delta v. Screenshot of .html output

image

@smiths
Copy link
Collaborator

smiths commented Mar 15, 2019

@oluowoj, this final equation looks good to me. What branch is this in? I went to have a look at the current documentation, but I want to make sure I'm in the right branch. Is it gamephy_SRS?

@oluowoj
Copy link
Collaborator Author

oluowoj commented Mar 15, 2019

yes, that's correct.

@smiths
Copy link
Collaborator

smiths commented Mar 15, 2019

Thank you @oluowoj. I had a quick look at the new documentation. It looks to be going in the right direction to me. I think we are safe to close this issue now.

@JacquesCarette
Copy link
Owner

I re-opened the issue because while the equation is now correct, the fix loses information.

@Awurama-N
Copy link
Contributor

I was looking for the pull request that these changes were suppose to be made in and i found PR #1376 , I saw that you brought up the fact that things were going missing in the comment and the PR was closed and never merged. But I went through the changes made and compared it to the current state of the files that were suppose to be changed, I see some of the changes might have been made in other PR's since they exist now but others don't.
Would you like me to go through any differences and replicate some of the changes to resolve this issue without any other information disappearing.

@JacquesCarette
Copy link
Owner

Yes please.

@balacij balacij added needs-action-items A clear 'path to resolution' is needed to close any ticket. needs-clarification Needs a clear 'state', 'goal', 'analysis', and 'explanation' to reduce solution ambiguities. labels Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-action-items A clear 'path to resolution' is needed to close any ticket. needs-clarification Needs a clear 'state', 'goal', 'analysis', and 'explanation' to reduce solution ambiguities.
Projects
None yet
Development

No branches or pull requests

5 participants