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

Subsector comparison to MPCA state inventory #86

Open
wants to merge 798 commits into
base: main
Choose a base branch
from

Conversation

pawilfahrt
Copy link
Collaborator

@pawilfahrt pawilfahrt commented May 31, 2024

Checklist

Please complete this checklist as a courtesy to the PR reviewer.

Code and styling

  • All of the files/scripts I added are in the right place and named appropriately. See the README for details.
  • I have not used setwd()
  • I have used file.path(here::here(), "file_name")) to source any scripts or read in data
  • I have not added any large datasets, unless absolutely necessary (explain)
  • I have commented my code, particularly in hard to understand areas
  • I have added additional package dependencies as necessary with renv::install()
  • I have run styler::style_dir(".", recursive = TRUE, filetype = c("R", "qmd"))
  • Plots
    • If plotly, use source = "chunk-name" in plot_ly()
    • Use formatting with councilR::plotly_layout()?
  • Chunk formatting
    • All chunks named
    • All figure or table chunks have caption
    • out.width: "95%". If a specific height is needed, use pixels. and/or out.height: "500px"

Document editing

  • I have ensured that modified documents knit successfully from render_for_publication.R
  • I have fixed any missing citations, cross references, hyperlinks
  • I have reviewed my contributions for typos and misspellings.

GitHub and project management

  • I have identified and assigned at least one Reviewer (Liz, Sam, Laine) to this PR
  • I have assigned myself to this PR
  • I have updated the status in the GitHub Project

@pawilfahrt pawilfahrt added the meta project-level work label May 31, 2024
@pawilfahrt pawilfahrt requested a review from eroten May 31, 2024 20:59
@pawilfahrt pawilfahrt self-assigned this May 31, 2024
@pawilfahrt pawilfahrt requested a review from ksmiff33 as a code owner May 31, 2024 20:59
@pawilfahrt pawilfahrt linked an issue May 31, 2024 that may be closed by this pull request
@pawilfahrt
Copy link
Collaborator Author

There are still some lingering formatting fixes to make on my end

@eroten eroten changed the title 85 subsector comparison to mpca state inventory Subsector comparison to mpca state inventory Jun 27, 2024
@eroten eroten changed the title Subsector comparison to mpca state inventory Subsector comparison to MPCA state inventory Jun 27, 2024
pawilfahrt and others added 30 commits October 29, 2024 16:05
adds globals, listenv, furrr, future, parellely, codetools
…missions-process-types

Transportation - Integrate all emissions process types
these were previously errantly overwritten on dev-2005-baseline.

Recovered necessary MN scripts from those but these should be double checked.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta project-level work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Subsector comparison to MPCA state inventory
3 participants