forked from pnxenopoulos/awpy
-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Initial Map Control Calculations/Visualizations and Testing (pnxenopo…
…ulos#235) * Initial Map Control Calculations/Visualizations and Testing * Addressed Jan Eric's PR comments, Added more documentation in notebook, Re-named some functions for code clarity purposes * Fixed pre-commit errors, added opencv-python to requirements.txt, fixed notebook documentation * pre-commit and pyright fixes * Cherry-picked commits from Jan Eric's map_control branch which was a working, merged version of my branch. Added extra fix for black. Addressed Jan-Eric's PR comments as well; highlights below: * Added numerous dataclasses for map control purposes in types.py * Removed uneccessary error checking * Updated docstring documentation and formatting * Updated test_map_control.py * Addressed most of Jan Eric comments in plot.py * Addressing Jan Erics comments on map_control.py, plot.py, and types.py * Addressing rest of Jan Eric non-test PR comments * Addressing Jan Erics non-testing comments on PR * Addressed map cnotrol testing coverage issues. Added test in test_nav for calculate_tile_area. Fixed other code errors I came across as well * Addressing unresolved comments not related to visualization testing * Changes to types.py after rebasing main branch updates * Changing function names to match awpy naming convention * Removing deprecated functions, cleaning up notebook, adding testing for new plotting functions * Addressing issues with new test_vis map control function * Slight change to ommit returned axes object from plot_frame_map_control in plot_round_map_control * Adding coverage in tests, removing opencv from requirements * Addressing test failures in test_plot_round_map_control and test_calc_frame_map_control_values * BFS now terminates based on cumalative area of tiles instead of number of steps from source tile. Updataes were made to the tests as well to reflect changes in codebase
- Loading branch information
1 parent
db9d275
commit d70a66e
Showing
10 changed files
with
1,475 additions
and
23 deletions.
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,3 @@ | ||
"""Provides data visualization capabilities for CSGO data.""" | ||
SIDE_COLORS = {"ct": "#5d79ae", "t": "#de9b35"} | ||
AWPY_TMP_FOLDER = "csgo_tmp" |
Oops, something went wrong.