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

[tool] Update check_overmap_coverage.py #2427

Merged
merged 14 commits into from
Jun 28, 2024

Conversation

vetall812
Copy link
Contributor

@vetall812 vetall812 commented Jun 25, 2024

Summary

Changed Camel to Snake name style
Changed exit() to raise()
Added -y | --yes key to remove user input after execution
Added ability to select on of predefined marking symbol -m | --mark
Added ability to sort results -s | --sort
Added ability to export result to csv file -f | --file
Added filtering by name -n | --name
or by id -i | --id
Added ability to print file that have sprited ID -j | --json

Content of the change

Testing

different types of filtering:

image
image
image

export results to csv:

image
image

last example execution command:
check_overmap_coverage.py . SurveyorsMap -m 'v' -s percent -i aband -n ware -f aban_ware.csv

(The dot here reflects my unwillingness to specify the path to the game; I want the script to find it automatically.)

image

Additional information

@vetall812 vetall812 changed the title [tool] Update check_overmap_coverage.py so it is styled as common python script [tool] Update check_overmap_coverage.py Jun 27, 2024
@Fris0uman Fris0uman merged commit 0653a91 into I-am-Erk:master Jun 28, 2024
@vetall812 vetall812 deleted the tool_check_OM_coverage_fix_style branch August 24, 2024 18:46
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