Skip to content
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

Refinery cannot handle natural gas import/export for both NL and GER #51

Closed
Richard-Deuchler opened this issue Sep 4, 2013 · 19 comments
Assignees

Comments

@Richard-Deuchler
Copy link

As discovered in #50, refinery currently cannot deal with the fact that Germany is a net importer of natural gas in the present scenario. This is because we hard-coded the demand of the energy_import_natural_gas converter to 0.0.

It is now not possible to make Germany a net importer of natural gas.
screen shot 2013-09-04 at 10 55 31 am
(note: this is a screeenshot of what it will look like after we fixed the bug with the unknown biogas demand in the German scenario. The fix is not yet pushed/merged but will be soon).

Because the extraction of natural gas is not sufficient, and import is set to 0.0, there is not enough supply of natural_gas. The converter energy_natioanal_gas_network_gas does not receive enough energy.

including @antw

@ghost ghost assigned ChaelKruip Sep 4, 2013
@Richard-Deuchler
Copy link
Author

To further illuminate:
When not hard-coding the demand of import_natural_gas, the NL graph is incomplete and looks like this:

screen shot 2013-09-04 at 11 14 22 am

Refinery does not understand that energy that energy_natioanal_gas_network_gas is lacking should be satisfied with natural_gas.

@antw
Copy link
Contributor

antw commented Sep 4, 2013

Refinery does not understand that energy that energy_natioanal_gas_network_gas is lacking should be satisfied with natural_gas.

@Richard-Deuchler Are you sure you have the very latest version of Refinery? I made changes this week which should have made exactly that sort of calculation possible. In your Atlas directory, please git up and bundle install.

If that doesn't solve it, let me know.

@wouterterlouw
Copy link

Note that energy_national_gas_network_natural_gas is in fact a pseudomixer. If it was just a normal node with incoming edges of the same carrier, Refenery was most likely able to calculate it! See also quintel/etsource#498.

@antw
Copy link
Contributor

antw commented Sep 4, 2013

I just tried this myself, and @Richard-Deuchler is right; the energy_treatment_natural_gasenergy_national_gas_network_gas edge doesn't get calculated. I'll look into it and find out why.

@antw
Copy link
Contributor

antw commented Sep 4, 2013

I'll look into it and find out why.

My changes could calculate the natural gas edge, but only if we know the demand of both the treatment node and the network node; which we do only if we already know the import demand.

@wouterterlouw is also right; if the greengas distribution node were outputting natural gas instead (so that the network node had only one incoming carrier), Refinery will deal with that perfectly.

Fortunately, I can hack something together to deal with this in the short-term:

Initial values Calculated
_inatgas _natgas

@ChaelKruip
Copy link

Fortunately, I can hack something together to deal with this in the short-term:

👍

@ghost ghost assigned antw Sep 4, 2013
@ChaelKruip
Copy link

Assigning @antw

@StijnDellaert
Copy link

Note that, although it is invisible with the currently incalculable nodes, the uranium distribution/treatment/import/export will fail validation for Germany since extraction is higher than use, but the import and export are linked to different converters.

Any chance the fix you're making will also fix this @antw ? (it's this issue #44 I think, it's on hold for now)

antw added a commit that referenced this issue Sep 4, 2013
Adds a way to fill in the remaining demand for a node, by assigning it
to an edge even if the node has multiple input carriers.
antw added a commit to quintel/atlas that referenced this issue Sep 4, 2013
@antw
Copy link
Contributor

antw commented Sep 4, 2013

I have pushed the Refinery changes. Please git up and bundle install in your Atlas directory. The fix works specifically for this arrangement of nodes: a mixer with multiple input carriers, we know the demands of all but one incoming edge (i.e. we know the greengas input to the mixer). Over the next few days I'll clean this up and improve the way Refinery deals with nodes which have multiple carriers.

@Richard-Deuchler Does this fix the issue to your satisfaction (for the moment, at least)?

@StijnDellaert
Copy link

Nice work Anthony, the natural gas part is working now for DE and NL, even without the ugly hard-coded import/export.
Shall I push the etsource update (removing ~demand = 0 from energy_import_natural_gas)?

@ChaelKruip
Copy link

the uranium distribution/treatment/import/export will fail validation for Germany since extraction is higher than use, but the import and export are linked to different converters.

@StijnDellaert Did you check this as well? Does it work?

@antw
Copy link
Contributor

antw commented Sep 4, 2013

Did you check this as well? Does it work?

@Richard-Deuchler said to me on Skype that this is still a problem.

@StijnDellaert
Copy link

No, we get the validation error now, but this was described in #44

@antw
Copy link
Contributor

antw commented Sep 4, 2013

Can the uranium import/export be moved to the same node, was was done with natural gas? Long-term we want to support multi-node import export in Refinery (as described in #44), but there's pretty much no chance it will be done in time to use with this month's deploy.

@ChaelKruip
Copy link

After talking to @AlexanderWirtz, I propose to remove the energy_treatment_uranium_oxide node. This will effectively connect the import and export to the same node.

@Richard-Deuchler @StijnDellaert @wouterterlouw any objections?

@Richard-Deuchler
Copy link
Author

I am ok with it. 👍

@StijnDellaert
Copy link

No objections

@wouterterlouw
Copy link

@ChaelKruip 👍

@ChaelKruip
Copy link

Done. Closing.

antw added a commit to quintel/atlas that referenced this issue Dec 13, 2013
Richard-Deuchler added a commit to quintel/etsource that referenced this issue Dec 17, 2013
Removing the hard-coded natural gas import from etsource. After @antw adapted atlas (see quintel/refinery#51), this hard-coded demand is no longer required.
dennisquintel pushed a commit to quintel/etsource that referenced this issue Dec 20, 2013
Removing the hard-coded natural gas import from etsource. After @antw adapted atlas (see quintel/refinery#51), this hard-coded demand is no longer required.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants