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

Init #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Init #2

wants to merge 2 commits into from

Conversation

pkopka
Copy link

@pkopka pkopka commented Jun 6, 2019

  • Script calculates image quality statistics
  • Test image in iterfile format

* Script calculates image quality statistics
* Test image in iterfile format
Copy link

@K4liber K4liber left a comment

Choose a reason for hiding this comment

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

I marked some spelling bugs, but the script seems to work correctly.


parser.add_option("-i", "--input",
action="store", dest="input", metavar="PATH", type="string",
help="Path out file png")
Copy link

Choose a reason for hiding this comment

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

parser.add_option("-i", "--input",
action="store", dest="input", metavar="PATH", type="string",
help="Path out file png")
parser.add_option("-o", "--out",
action="store", dest="output", metavar="PATH", type="string",
help="Path input header interfile")

Param help is mismatched in the parser options.

Copy link
Author

Choose a reason for hiding this comment

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

Add help to the parser options

image_parameters/image_quality_stats.py Outdated Show resolved Hide resolved
def interfile2array(param):
"""
conveft interfile to numpy array
:param param: dict contens interfile poarametrs
Copy link

Choose a reason for hiding this comment

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

conveft interfile to numpy array
:param param: dict contens interfile poarametrs

conveft -> convert
contens intefile poarametrs -> containing interfile parameters

Copy link
Author

Choose a reason for hiding this comment

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

done

conveft interfile to numpy array
:param param: dict contens interfile poarametrs
:return:
"""
Copy link

Choose a reason for hiding this comment

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

def txt2array(file_names, pram_dict, filip=False):
"""
conveft interfile to numpy array
:param param: dict contens interfile poarametrs
:return:
"""

Not exactly description of this function. This function is never used.

Copy link
Author

Choose a reason for hiding this comment

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

Function has been removed.

return arr


def txt2interfile(file_names, pram_dict, ouput_name='test.v'):
Copy link

Choose a reason for hiding this comment

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

def txt2interfile(file_names, pram_dict, ouput_name='test.v'):
This function is never used.

Copy link
Author

Choose a reason for hiding this comment

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

Function has been removed.

image_parameters/image_quality_stats.py Outdated Show resolved Hide resolved
image_parameters/image_quality_stats.py Outdated Show resolved Hide resolved
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