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

delete realization complex of module 35_transport #1543

Merged

Conversation

LaviniaBaumstark
Copy link
Member

@LaviniaBaumstark LaviniaBaumstark commented Feb 6, 2024

Purpose of this PR

cleaning: delete realization complex of module 35_transport and all related code parts

Type of change

  • Refactoring

Checklist:

  • My code follows the coding etiquette
  • I performed a self-review of my own code
  • I explained my changes within the PR, particularly in hard-to-understand areas
  • All automated model tests pass (FAIL 0 in the output of make test)
  • The changelog CHANGELOG.md has been updated correctly

Further information (optional):

  • Test runs are here:
  • Comparison of results (what changes by this PR?):

core/sets.gms Outdated
@@ -343,6 +331,14 @@ $ENDIF.WindOff
seceaf "Route: Scrap-loaded EAF"
pcc "outdated technology, only here to avoid compilation errors if input data containing information for this technology are used"
pco "outdated technology, only here to avoid compilation errors if input data containing information for this technology are used"
*** transport technologies for deleted realization complex of module 35_transport - only here for input data consistency
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe not "input data consistency", but rather "to make it possible to process input data that still includes data for these obsolete transport technologies"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed

core/sets.gms Show resolved Hide resolved
main.gms Outdated
@@ -1448,18 +1437,18 @@ $setglobal cm_incolearn off !! def = off
$setglobal cm_storageFactor off !! def = off
*** cm_learnRate "change learn rate value by technology."
*** def <- "off" = no change for learn rate value;
*** or list of techs to change learn rate value. (ex. "apcarelt 0.2")
*** or list of techs to change learn rate value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would keep an example here, but use another tech, e.g. spv

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please provide a good example. I don't know one

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(ex. "spv 0.2")

main.gms Outdated
$setglobal cm_learnRate off !! def = off
*** cm_adj_seed and cm_adj_seed_cont "overwrite the technology-dependent adjustment cost seed value. Smaller means slower scale-up."
*** both swicthes have the same functionality, but allow more changes once the character limit of cm_adj_seed is reached.
*** def <- "off" = use default adj seed values.
*** or list of techs to change adj_seed value. (ex. "apCarH2T=0.5,apCarElT=0.5,apCarDiEffT=0.25,apCarDiEffH2T=0.25")
*** or list of techs to change adj_seed value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

again, I would keep an example using other technologies.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please provide a good example. I don't know one

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(ex "spv=1, tnrs = 0.1")

main.gms Outdated
$setglobal cm_adj_seed off
$setglobal cm_adj_seed_cont off
*** cm_adj_coeff and cm_adj_coeff_cont "overwrite the technology-dependent adjustment cost coefficient. Higher means higher adjustment cost."
*** both swicthes have the same functionality, but allow more changes once the character limit of cm_adj_coeff is reached.
*** def <- "off" = use default adj coefficient values.
*** or list of techs to change adj_coeff value. (ex. "apCarH2T=100,apCarElT=100,apCarDiEffT=200,apCarDiEffH2T=200")
*** or list of techs to change adj_coeff value.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

again, I would keep an example using other technologies.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(ex. "gash2=1, hydro=0.1")

modules/35_transport/edge_esm/sets.gms Show resolved Hide resolved
@LaviniaBaumstark LaviniaBaumstark merged commit 3c1f7b8 into remindmodel:develop Feb 8, 2024
2 checks passed
@LaviniaBaumstark LaviniaBaumstark deleted the cleaningRealizations branch May 3, 2024 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants