Matt Jamieson

Things like coal and natural gas rollups should be of type system_process because they are rollups of other systems. That being said, it's only preferred. I don't think we should...

I was going to say these seem like things I should've implemented in [issues/150](https://github.com/USEPA/ElectricityLCI/tree/issues/150), but I forgot about them it seems. There's probably a thought here that implementing these commented...

> @m-jamieson, from what I can tell, you don't need the two `JOIN` statements in the second query, right? They were added in the first query just for the return...

> I know internal IDs on exchanges are just a clerical thing, but do we know if there any concern if the exchange list IDs are non-consecutive? Cause this is...

You happen to try this at all? Looking at the example, there is a lot that goes on inside that module with almost nothing inside functions. I *think* the most...

The ultimate intent here is to have UUIDs generated using the namespace of the process - in this instance, the namespace would be the total "path" to that unit process...

Possibly related to #190

This isn't a direct answer to your question, but if you're truly just interested in obtaining the openLCA json-ld zip file from running ELCI_1 - you could get the pre-packaged...

Your best chance at generating something new will be using the [development branch](https://github.com/USEPA/ElectricityLCI/tree/development). I've been working to reduce the dependencies on eGRID there that will limit the master branch code....

It doesn't appear all that important now. In earlier iterations of the data, coal and natural gas upstream dataframes weren't formatted correctly, so they required some columns, etc. added so...