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

[generators]10_correct_or_clarify #62

Merged
merged 1 commit into from
Oct 27, 2020
Merged

[generators]10_correct_or_clarify #62

merged 1 commit into from
Oct 27, 2020

Conversation

shlff
Copy link
Member

@shlff shlff commented Oct 16, 2020

Hi @jstac , this PR corrects the following possible typo in lecture generators:

  • \lL(\ell_1)(L(l_1)) -->> \linop (L(B))

Even if \lL(\ell_1) is correct here, since it appears without defining first (we define \lL(\ell_1) in the next lecture uc_mc_semigroups), we'd better add a clarification.

@@ -223,7 +223,7 @@ when $S$ is finite, we introduced the derivative of a map $t
The derivative was defined by differentiating $P_t$ element-by-element.

This coincides with the operator-theoretic definition in {eq}`devlim` when $S$
is finite, because then the space $\lL(\ell_1)$ is finite dimensional, and
is finite, because then the space $\linop$ is finite dimensional, and
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks @shlff -- please change this to "...the space $\lL(\ell_1)$, which consists of all bounded linear operators on $\ell_1$, is..."

@shlff
Copy link
Member Author

shlff commented Oct 20, 2020

Thanks for your nice advice, @jstac . Sorry for my late reply.

I made the change accordingly and this PR is ready to merge.

@jstac
Copy link
Collaborator

jstac commented Oct 27, 2020

Thanks @shlff

@jstac jstac merged commit 7b6497d into master Oct 27, 2020
@jstac jstac deleted the sg10 branch October 27, 2020 23:52
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.

2 participants