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

erratum: chapter 4 #6

Closed
khosrogh opened this issue Dec 12, 2021 · 2 comments
Closed

erratum: chapter 4 #6

khosrogh opened this issue Dec 12, 2021 · 2 comments

Comments

@khosrogh
Copy link

  • Page 83: in the numbered list, at number 2, Mapping' definition, it reads: "Any operation that can run independently on a split is called a map. In the preceding diagram, the map operation coverts ..."
    I think there is a typo for the word coverts should be converts. Would you please check this?

  • Page 86: in the greedy algorithm defining list number 2, it is talked about Union(S, e). I couldn't find out what is e?
    A bit of definition would suffice.

  • Page 88: do the code in the middle of the page is complete?
    I think there are at least two misplaced lines of code and need changes:

for i in range(len(aTour)))
and
for c in range(number_of_cities))
@rajat-packt
Copy link
Contributor

rajat-packt commented Dec 30, 2021

Hey @khosrogh, sorry for such long delay.
We were finally able to confirm with the author, all your suggestions have been valid and will we update the readme file to reflect the same.

As for the missing definition on Page 86, the author has provided this explanation:

Union(S,e)
e is the delta corresponding to the element k that we are adding to converge the solution

With this we'll be closing all the issues after updating the readme.
Thanks for reaching out to us!

@khosrogh
Copy link
Author

Hi @rajat-packt,

It's OK. Anytime. <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants