Skip to content

Conversation

@nathanhuynhers
Copy link
Collaborator

@nathanhuynhers nathanhuynhers commented May 1, 2025

What's new in this PR

Description

  • fixed issue where min and max values of histogram did not reflect in modal when a non-project size filter was cleared
  • fixed issue where location dropdown's apply and clear buttons were active even when nothing was clicked
  • fixed issue where histogram was uneven and solved by using log10 function
  • ensured that region, county, and town filtering are not case sensitive
  • fixed bug where county and town filtering didn't work for projects in more than 1 county or town
  • removed "test" from the db table names
  • added link to ace ny's website and allowed cursor to appear while hovering over ace ny and email links infomodal
  • fixed issue where cod and kdm dates were off by one due to time zone conversions
  • utility service territory data changed so adjusted filtering logic
  • fixed energy and pumped storage filtering bug

Screenshots

How to review

  • test clearing the filters and seeing max/min values in histogram

Next steps

Relevant links

Online sources

Related PRs

CC: @jjstnlee

@nathanhuynhers nathanhuynhers linked an issue May 1, 2025 that may be closed by this pull request
@ethan-tam33 ethan-tam33 force-pushed the nathan/186-project-size-histogram-w-all-projects branch from 4716a75 to b1f52c7 Compare May 20, 2025 03:44
@ethan-tam33 ethan-tam33 changed the title Evening out histogram, even when there are large outliers [bug] fixed location and project size dropdown bugs May 20, 2025
@ethan-tam33 ethan-tam33 requested a review from jjstnlee May 20, 2025 03:49
@ethan-tam33 ethan-tam33 force-pushed the nathan/186-project-size-histogram-w-all-projects branch from ebff8d0 to d475fac Compare May 22, 2025 05:10
jjstnlee and others added 2 commits May 21, 2025 23:38
…thub-personal:calblueprint/ace-ny into nathan/186-project-size-histogram-w-all-projects
Copy link
Collaborator

@jjstnlee jjstnlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GREAT JOB

@jjstnlee jjstnlee merged commit e599d7c into main May 22, 2025
2 checks passed
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.

Project Size Histogram w/ All Projects

4 participants