-
Notifications
You must be signed in to change notification settings - Fork 6
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
V3 ng production proxy #56
Open
haclohman
wants to merge
34
commits into
main
Choose a base branch
from
v3_ng_production_proxy
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
includes gb_stations_proxy, storage_comp_station_proxy, trans_comp_station_proxy
start of the natural gas production proxy using enverus prism and di data
Complete code for oil and natural gas exploration and production proxies
Separating out ng_storage_comp_station_proxy and ng_trans_comp_station_proxy from this code.
Added Header. Made structural changes. Removed unused packages.
Added Header. Removed unused imports. Adjusted enverus_production_path to exist in pathway. Noted that there is a #TODO statement, but left because I was unsure if it was accurate. There are some unused variables identified in the script.
Added Header. Removed unused packages. Made structural adjustments.
Added Header. Removed unused imports. Added structural edits.
Added Header. Removed unused imports. Adjusted structure.
Added Header. Removed unused imports. Made aesthetic changes.
Added Header. Removed unused imports. Cleaned structure.
Added Header. Removed unused imports. Adjusted structure.
Fixed Header.
Fixed Header
Fixed Header
Fixed Header
Fixed Header
Fixed Header. Removed unused imports.
Fixed purpose in header
Fixed purpose in header.
Added Header. Removed unused imports.
Added Header. Removed Unused imports.
Added Header. Removed Unused packages. Edited structure.
Added Header. Removed unused libraries.
Added Header. Removed unused packages.
Filled in missing data for state-year pairs for oil and gas proxies missing data.
added annual_rel_emi for monthly natural gas and petroleum proxies.
Remove temp file sums.csv
Fixing year_month issue in oil_federal_gom_offshore_proxy
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Natural gas and petroleum exploration and production proxies ready for review. ng_oil_production_utils.py is a file that contains functions used across all proxies and task_enverus_di_prism_data_processing.py is the script that processes and cleans up the Enverus DI and Prism data that is stored on disk and read into all the proxies. It might make sense to review these two scripts first and then review the individual proxies.