Greg Miller

Results 159 comments of Greg Miller

Great idea. PUDL uses CI for their repo: https://catalystcoop-pudl.readthedocs.io/en/latest/dev/testing.html?highlight=continuous%20integration#continuous-integration-tests Catalyst actually published a template that we could consider using for setting this up: https://github.com/catalyst-cooperative/cheshire Separate issue, but I think we...

I do not have any context on these two fields.

Digging into this further, I started looking into plant `54809`, which the validation check shows missing some fuel consumption data, but not net generation data. This plant has 6 IC...

Ok it turns out that one issue was that when I was comparing inputs to outputs, I was comparing the outputs to `generation_fuel_eia923` table, which I didn't realize excluded nuclear...

Just wanted to bump this issue: we're having some issues with bad timezone data due to bad lat/long values (https://github.com/catalyst-cooperative/pudl/issues/1192). We're going to try and patch this on our end,...

Linking this to https://github.com/catalyst-cooperative/pudl/issues/971 and https://github.com/catalyst-cooperative/pudl/issues/402

We are also noticing this issue for Pegasus Wind (plant ID 61916), which in EIA-860 is listed with a coordinate of: ``` latitude 43.452003 longitude -83.50721 ``` Which correctly places...

After diving into the raw EIA-860 tables a bit more, it looks like in some of the earlier years (eg 2017) for plant 61916, they were incorrectly reporting a longitude...

After looking into the math a bit more, it looks like perhaps the input value of `x` is in per unit on system base quantities, and the calculated output of...

I'll mention that a lot of our code involves some steps that just deal with converting the data back and forth between the format used by gridemissions, since we also...