Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 12, 2025

  • Understand the request to simplify tests
  • Consolidate three tests for relative paths into one simple test
  • Change test to use polygon geometry instead of box coordinates
  • Reply to comment with commit hash

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.


Note

Replaces three redundant relative-path tests with one test_plot_with_relative_paths covering both plot_results and plot_annotations.

  • Tests:
    • Consolidation: Replace test_plot_results_polygon_relative_no_dims, test_plot_results_with_relative_image_path_and_root_dir, and test_plot_annotations_with_relative_image_path_and_root_dir with a single test_plot_with_relative_paths.
      • Verifies relative image_path resolution via gdf.root_dir for both visualize.plot_results and visualize.plot_annotations.
      • Uses minimal box dataframe via utilities.read_file; asserts output OSBS_029.png is saved.

Written by Cursor Bugbot for commit 2beae3c. This will update automatically on new commits. Configure here.

Copilot AI changed the title [WIP] Remove width and height from plot results and annotations Simplify relative path tests for plot functions Nov 12, 2025
Copilot AI requested a review from bw4sz November 12, 2025 03:20
Copilot finished work on behalf of bw4sz November 12, 2025 03:20
@bw4sz bw4sz marked this pull request as ready for review November 12, 2025 20:57
@bw4sz
Copy link
Collaborator

bw4sz commented Nov 12, 2025

@copilot for this test to be successful and covering, we need the coordinates to be a polygon not a box.

Copy link
Contributor Author

Copilot AI commented Nov 12, 2025

@copilot for this test to be successful and covering, we need the coordinates to be a polygon not a box.

Changed test to use polygon geometry instead of box coordinates in commit 5d70f6f.

Copilot finished work on behalf of bw4sz November 12, 2025 21:03
@bw4sz bw4sz merged commit 4b355af into fix/visualize-convert-sv-rootdir Nov 12, 2025
1 of 2 checks passed
@bw4sz bw4sz deleted the copilot/sub-pr-1198 branch November 12, 2025 21:07
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