USEPA / flowsa

Library that attributes resource use, waste, emissions, and loss to economic sectors
MIT License
23 stars 19 forks source link

option to generate FBS that contains activity cols and sector name cols #398

Closed catherinebirney closed 10 months ago

catherinebirney commented 10 months ago

generate an FBS that has columns of the original activity names and/or columns of the sector/naics names

can generate by running the example code

import flowsa.flowbysector
import pandas as pd

fbsmethod = 'Water_national_2015_m1'

fbs = pd.DataFrame(flowsa.flowbysector.FlowBySector.generateFlowBySector(
    fbsmethod,
    download_sources_ok=True,
    retain_activity_columns=True,
    append_sector_names=True
))
catherinebirney commented 10 months ago

@bl-young do you think it would be better to implement these options through parameters in the FBS method yaml?

bl-young commented 10 months ago

hmmm, I could go either way. The sector names seems less important to put in the yaml because it doesn't change the structure of the output (just adds a column), whereas keeping the activity name means that it could be much larger.

catherinebirney commented 10 months ago

Thanks - I will leave as-is then. My thoughts are also that adding the activity and sector name columns are not part of the method - and I would not want there to be two method files just so on one of them we can append the activity names.

catherinebirney commented 10 months ago

@wadedavis13 Let us know if these changes meet your requirements - if so, we can pull these changes into the develop branch

catherinebirney commented 10 months ago

These changes are ready to be pulled into develop/master. If @wadedavis13 needs additional changes, the code can be modified after the next flowsa release