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

Add new write result function #17

Conversation

timeforplanb123
Copy link

write_result:

  • function writes Result object to file
  • write_result returns diff between old file and new file (str)
  • parameters:
    • filename - file you want to write the result. write_result creates dirs with filename if it's necessary
    • vars, failed, severety_level
    • write_host - write hostname to file or not
    • count is the same that count from print_result. It's number of sorted results (-5 or 5, for example). It can be useful for a large number of identical results. It's acceptable to use numbers with minus sign(-5 as example), then results will be from the end of results list.
    • append - "a+" or "w+" mode
    • no_errors - dont't write result with errors to file

add new count parameter. It's number of sorted results. It can be useful for a large number of identical results. It's acceptable to use numbers with minus sign(-5 as example), then results will be from the end of results list.
print_stat prints statistic for Result object
write_result writes Result object to file
add new count parameter to print the number of results. Useful for a lot of identical results
print_stat prints statistic for Result object
write_result writes Result object to file
add new count parameter to print the number of results. Useful for a lot of identical results
print_stat prints statistic for Result object
write_result writes Result object to file
@dbarrosop
Copy link
Contributor

Same as with other PRs, commits are mixed somehow

@timeforplanb123
Copy link
Author

Replaced all new functions to single PR with ipynb examples #21

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