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

Move plots and features in the spot check integration script into codebase #116

Closed
2 tasks
BainanXia opened this issue May 27, 2020 · 0 comments · Fixed by #129
Closed
2 tasks

Move plots and features in the spot check integration script into codebase #116

BainanXia opened this issue May 27, 2020 · 0 comments · Fixed by #129
Assignees
Labels
Epic new feature Feature that is currently in progress.

Comments

@BainanXia
Copy link
Collaborator

BainanXia commented May 27, 2020

Spot check script integration is finished in #264
File directory: Dropbox (IVL)/Explorations/BainanX/spot check script integration/

Researchers are going to use the integrated script for scenario post analysis in short term. In the meantime, we would like to gradually move the features that are implemented in the script into the code base together with corresponding tests.

Before doing so, we have following tasks to clear:

  • A big picture design of the architecture and interface of data/calculations/tables/plots in PostREISE, so that the code performance in terms of time and memory space is optimized.
  • Refactor existing code, such as AnalyzePG to follow the rules we set up above.

Additional comments repost during the presentation on 05/27/2020:
@danielolsen : (12) might be easier to interpret with a log-scale X-axis
@danlivengood : It would be great if the plots (applicable bars, scatters, etc) could have the capabilities to sort along axes, such as states in alphabet order, which is currently based on the default order in our grid (Eastern, Western, Texas)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic new feature Feature that is currently in progress.
Projects
None yet
8 participants