You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description: willScanNewCells() passes an Aircraft by value rather than by reference in order to use the Aircraft's scan() function to determine if new Cells will be scanned. This could be a cause of the searching performance on large GridMaps. A 100x100 Grid means the Aircraft takes >240KB of memory, which is created each time willScanNewCells() is called in findRoute() (which is three times per loop). Modifying willScanNewCells() to allow passing by reference could possible improve performance. Priority: Low User Story: As a user, I want the router to run efficiently in terms of memory and time. Acceptance Criteria: Change willScanNewCells() as described and compare the run time on the 100x100 grid unit test. If there is a >3% improvement on the unit test's reported run time after the change, keep the change and merge to development.
The text was updated successfully, but these errors were encountered:
Assigned Dev: N/A
Description: willScanNewCells() passes an Aircraft by value rather than by reference in order to use the Aircraft's scan() function to determine if new Cells will be scanned. This could be a cause of the searching performance on large GridMaps. A 100x100 Grid means the Aircraft takes >240KB of memory, which is created each time willScanNewCells() is called in findRoute() (which is three times per loop). Modifying willScanNewCells() to allow passing by reference could possible improve performance.
Priority: Low
User Story: As a user, I want the router to run efficiently in terms of memory and time.
Acceptance Criteria: Change willScanNewCells() as described and compare the run time on the 100x100 grid unit test. If there is a >3% improvement on the unit test's reported run time after the change, keep the change and merge to development.
The text was updated successfully, but these errors were encountered: