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
Fix documentation misprint, modulus should be norm. #14367
Conversation
2df4370
to
ba81a01
Compare
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.
Realized I missed something,,, Converted to draft in the meantime. |
* \int_{S\cap F} f |d\bar{r}| = | ||
* \int_{\hat{S}\cap\hat{F}} f \circ F_{K} | J d\hat{r}| | ||
* \approx \sum_{q} f \left(F_{K}(\hat{x}_{q}) \right) |J_q \hat{t}_q| w_q, | ||
* \int_{S\cap F} f \|d\bar{r}\| = |
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.
That's not quite right -- dr
is an area or volume element, and as such is a scalar. Depending on how you define the transformation, it may be positive or negative, and so using |...|
makes sense. But it isn't a vector.
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.
Sorry for not fixing this sooner... No, I really do mean the vector here, but I realized that my notation was strange, so I rewrote this. Updated now.
* \int_{\hat{S}\cap\hat{F}} f \circ F_{K} | J d\hat{r}| | ||
* \approx \sum_{q} f \left(F_{K}(\hat{x}_{q}) \right) |J_q \hat{t}_q| w_q, | ||
* \int_{S\cap F} f \|d\bar{r}\| = | ||
* \int_{\hat{S}\cap\hat{F}} f \circ F_{K} \| J d\hat{r} \| |
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.
This also applies here
* \approx \sum_{q} f \left(F_{K}(\hat{x}_{q}) \right) \|J(\hat{x}_q) | ||
* \hat{t}_q \| w_q, |
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.
But I think this here would then be correct.
Ping? |
ba81a01
to
455f7c8
Compare
OK, that makes sense. Thanks! |
No description provided.