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

[Sim_market] Only viable fields in get_configurable_fields #498

Open
NikkelM opened this issue Jun 8, 2022 · 0 comments
Open

[Sim_market] Only viable fields in get_configurable_fields #498

NikkelM opened this issue Jun 8, 2022 · 0 comments
Labels
- Prio 2 Should-Haves: Low-Priority bugs, QoL changes, Performance good first issue Not a bad second issue either! refactoring Make code look shiny

Comments

@NikkelM
Copy link
Collaborator

NikkelM commented Jun 8, 2022

As mentioned in #485, the linear and circular economies need different parameters in the config files. The current implementation contains all possible parameters for both. This should be changed to include only the valid parameters.
Make sure to add a new config file in the default_configuration-folder for this new distinction, and name them accordingly (some renaming might be necessary for places where the market-config is being loaded)

@NikkelM NikkelM added refactoring Make code look shiny - Prio 2 Should-Haves: Low-Priority bugs, QoL changes, Performance labels Jun 8, 2022
@NikkelM NikkelM added the good first issue Not a bad second issue either! label Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- Prio 2 Should-Haves: Low-Priority bugs, QoL changes, Performance good first issue Not a bad second issue either! refactoring Make code look shiny
Projects
None yet
Development

No branches or pull requests

1 participant