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 updates to config.py and layers.py in simfire/utils subpkg. #18

Conversation

afennelly-mitre
Copy link
Contributor

  • Update the LandfireLatLongBox to produce correct fuel/elev map shapes from H, W
  • Pull 'more' operational data than needed, then crop to calculated shape.

- Update the LandfireLatLongBox to produce correct fuel/elev map shapes from H, W
- Pull 'more' operational data than needed, then crop to calculated shape.
@afennelly-mitre afennelly-mitre self-assigned this Apr 29, 2024
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 29, 2024
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
simfire/game
   game.py1942388%48, 74–78, 245–246, 303–307, 336, 338, 340, 345, 348–355, 381, 394–395, 398–399
   image.py19289%20, 31
   sprites.py177697%79, 85, 90, 96, 328, 434
simfire/game/managers
   fire.py204598%223, 265–269, 643, 782
   mitigation.py44198%73
simfire/sim
   simulation.py41812271%73, 83, 93, 103, 113, 129, 140, 150, 170, 181, 324, 416–423, 457–478, 549, 615–628, 648–661, 679, 696, 768, 805–806, 808–809, 839, 858, 871, 892–959, 1017–1021, 1040–1051, 1065–1103
simfire/utils
   config.py46512673%68–81, 110, 212, 232, 246–253, 278–298, 309–316, 362–364, 382–384, 386–388, 407–418, 540, 555, 559, 563, 600–647, 666, 677–678, 681, 685, 695, 720, 726, 730, 741, 779–790, 813, 820, 828, 846–870, 964, 977, 1000, 1009–1030
   decorators.py13838%15–23
   generate_cfd_wind_layer.py867117%25–34, 38–56, 60–68, 72–82, 86–122, 126
   graph.py67691%107–108, 113, 119, 227, 252
   layers.py2695679%82, 86–87, 171, 189, 196, 234, 340–349, 352–393, 399–406, 589, 618
   log.py321069%11–12, 15–16, 21, 27, 36–38, 41
   terrain.py38879%127–136
   units.py25484%15–16, 29–31
simfire/utils/_tests
   test_layers.py89397%102, 131, 192
simfire/world
   elevation_functions.py24196%100
simfire/world/wind_mechanics
   cfd_wind.py1503974%63–73, 76–95, 130–134, 140–144, 188, 272, 274, 279, 281
   perlin_wind.py35197%42
   wind_controller.py663744%119–146, 157–170, 173, 176, 179, 182, 185
TOTAL363552985% 

Tests Skipped Failures Errors Time
97 0 💤 0 ❌ 0 🔥 7m 27s ⏱️

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 29, 2024
@doyled-it doyled-it merged commit 136623a into main Apr 29, 2024
6 checks passed
@doyled-it doyled-it deleted the 17-update-landfirelatlongbox-to-produce-expected-map-shapes-from-h-w branch April 29, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update LandfireLatLongBox to produce expected map shapes from H, W
2 participants