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

Added a reader for HUCM bins. #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bdolan44
Copy link

Added the ability to read in HUCM data to the dsd object. Entry point is read_hucm.

@pep8speaks
Copy link

Hello @bdolan44, Thank you for submitting the Pull Request !

Line 13:19: E231 missing whitespace after ','
Line 13:25: E231 missing whitespace after ','
Line 23:34: E231 missing whitespace after ','
Line 23:40: E231 missing whitespace after ','
Line 24:1: W293 blank line contains whitespace
Line 31:1: E303 too many blank lines (4)
Line 37:28: E231 missing whitespace after ','
Line 37:34: E231 missing whitespace after ','
Line 43:22: E225 missing whitespace around operator
Line 43:41: E231 missing whitespace after ','
Line 43:43: E231 missing whitespace after ','
Line 44:1: W293 blank line contains whitespace
Line 45:27: E231 missing whitespace after ','
Line 45:29: E231 missing whitespace after ','
Line 46:30: E231 missing whitespace after ','
Line 46:32: E231 missing whitespace after ','
Line 47:33: E231 missing whitespace after ','
Line 47:40: E231 missing whitespace after ','
Line 53:101: E501 line too long (103 > 100 characters)
Line 56:44: E231 missing whitespace after ','
Line 57:14: E111 indentation is not a multiple of four
Line 57:14: E117 over-indented
Line 57:40: E231 missing whitespace after ','
Line 58:14: E111 indentation is not a multiple of four
Line 96:11: E226 missing whitespace around arithmetic operator
Line 102:84: E231 missing whitespace after ','
Line 102:86: E231 missing whitespace after ','

Do see the Hitchhiker's guide to code style

@josephhardinee
Copy link
Owner

@bdolan44 Do you have (and willing to add) the script to take the HUCM output and put in the format that this expects so that others can use this?

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.

3 participants