search

arfc / pride

(P)lan for (R)ap(I)d (DE)carbonization
BSD 3-Clause "New" or "Revised" License
3 stars 7 forks source link

Updates fuel consumption in Temoa's model #138

Closed robfairh closed 3 years ago

robfairh commented 3 years ago

Fixes #136 and fixes #137.

I think I would leave the fuel consumption as MinActivity. If we specify an ExistingCapacity Temoa may deploy the vehicles but not use them. Also, ExistingCapacity was calculated dividing the MinActivity by 8760h, which gives ExistingCapacity the units of kgal/h.

robfairh commented 3 years ago

@samgdotson thoughts on this?

samgdotson commented 3 years ago

This is really a note for myself: we are defining the lifespan of vehicles by the lifespan of their most expensive part. We are assuming that the vehicle will be fully replaced rather than replacing a single, expensive, part. For fossil fuel vehicles, the most expensive part is the engine. For electric and fuel cells it's the battery.

samgdotson commented 3 years ago

It's not an issue with data_parser. Usually this means that Temoa failed to find a solution. What you've done is increase the minimum required activity, which now EXCEEDS the demand for those commodities. So, we have to update the demand for each year. I think it's fair to assume that the demand will remain the same for vehicle travel on and off campus. What do you think about that assumption?

On Mon, Jan 25, 2021 at 4:07 PM Roberto Fairhurst notifications@github.com wrote:

@robfairh commented on this pull request.

In temoa-uiuc/data_files/11_uiuc.sql https://urldefense.com/v3/__https://github.com/arfc/pride/pull/138*discussion_r564076264__;Iw!!DZ3fjg!tnYxnYGnWk0nnDl8E0hCeyiLGffrdLulsqWac4GT0QmHHPvXvskV5eVuxBIBSd6G$ :

@@ -238,9 +238,9 @@ INSERT INTO "MinActivity" VALUES('uiuc', 2024, 'IMPWIND', 23.35, 'electric GWh', INSERT INTO "MinActivity" VALUES('uiuc', 2025, 'IMPWIND', 23.35, 'electric GWh','must buy electricity that is produced'); INSERT INTO "MinActivity" VALUES('uiuc', 2026, 'IMPWIND', 23.35, 'electric GWh','must buy electricity that is produced');

-INSERT INTO "MinActivity" VALUES('uiuc', 2021, 'GSLVCL', 91.925, 'kgal',''); -INSERT INTO "MinActivity" VALUES('uiuc', 2021, 'DSLVCL', 20.9, 'kgal',''); -INSERT INTO "MinActivity" VALUES('uiuc', 2021, 'E85VCL', 16.4, 'kgal',''); +INSERT INTO "MinActivity" VALUES('uiuc', 2021, 'GSLVCL', 408.713, 'kgal',''); +INSERT INTO "MinActivity" VALUES('uiuc', 2021, 'DSLVCL', 116.828, 'kgal',''); +INSERT INTO "MinActivity" VALUES('uiuc', 2021, 'E85VCL', 25.727, 'kgal','');

@samgdotson https://urldefense.com/v3/__https://github.com/samgdotson__;!!DZ3fjg!tnYxnYGnWk0nnDl8E0hCeyiLGffrdLulsqWac4GT0QmHHPvXvskV5eVuxH2VNurq$ there is an issue with the simulations. If I use these new numbers for the MinActivity, the simulation produces empty figures. With the current numbers, it produces the figures fine. I believe this is something in data_parser.pyany ideas?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://urldefense.com/v3/__https://github.com/arfc/pride/pull/138*discussion_r564076264__;Iw!!DZ3fjg!tnYxnYGnWk0nnDl8E0hCeyiLGffrdLulsqWac4GT0QmHHPvXvskV5eVuxBIBSd6G$, or unsubscribe https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKSJ4WI6OYEJ4MHOHAGN4G3S3XTRHANCNFSM4WCL723A__;!!DZ3fjg!tnYxnYGnWk0nnDl8E0hCeyiLGffrdLulsqWac4GT0QmHHPvXvskV5eVuxLhO38AA$ .

-- Samuel G. Dotson Master of Science in Nuclear Engineering 2021 Bachelor of Science in Engineering Physics University of Illinois at Urbana-Champaign Office: 133 Computer Applications Building, 605 E Springfield Ave Pronouns: he/him/his sgd2@illinois.edu