Skip to content

Improves readability and processing time#4510

Merged
l3str4nge merged 2 commits into
TheAlgorithms:masterfrom
dpittaluga76:Fix-huffman-add-dict-printout
Jul 12, 2021
Merged

Improves readability and processing time#4510
l3str4nge merged 2 commits into
TheAlgorithms:masterfrom
dpittaluga76:Fix-huffman-add-dict-printout

Conversation

@dpittaluga76
Copy link
Copy Markdown
Contributor

@dpittaluga76 dpittaluga76 commented Jun 16, 2021

This removes lambdas, improves readability and processing time when it finds bitstring to print out.

  • Enhancement

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All functions and variable names follow Python naming conventions.

…when it finds bitstring to print out.

 Removes overuse of lambdas, uses dictionary instead.  This improves readability and processing time when it finds the bitstring to print out.
@ghost ghost added enhancement This PR modified some existing files awaiting reviews This PR is ready to be reviewed labels Jun 16, 2021
@dpittaluga76 dpittaluga76 changed the title Removes overuse of lambdas, improves readability and processing time Removes usage of lambdas, improves readability and processing time Jun 16, 2021
@ghost ghost added the tests are failing Do not merge until tests pass label Jun 20, 2021
@dpittaluga76 dpittaluga76 changed the title Removes usage of lambdas, improves readability and processing time Improves readability and processing time Jul 2, 2021
@ghost ghost removed awaiting reviews This PR is ready to be reviewed tests are failing Do not merge until tests pass labels Jul 12, 2021
@l3str4nge l3str4nge merged commit 2cbadc8 into TheAlgorithms:master Jul 12, 2021
@dpittaluga76 dpittaluga76 deleted the Fix-huffman-add-dict-printout branch July 14, 2021 20:47
shermanhui pushed a commit to shermanhui/Python that referenced this pull request Oct 22, 2021
* Removes overuse of lambdas, improves readability and processing time when it finds bitstring to print out.

 Removes overuse of lambdas, uses dictionary instead.  This improves readability and processing time when it finds the bitstring to print out.

* Update huffman.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement This PR modified some existing files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants