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

Implementation of #107 #135

Merged
merged 36 commits into from
Jul 5, 2021
Merged

Implementation of #107 #135

merged 36 commits into from
Jul 5, 2021

Conversation

Waldleufer
Copy link
Collaborator

@Waldleufer Waldleufer marked this pull request as draft June 23, 2021 12:49
SaiVarunVaranasi and others added 2 commits June 29, 2021 17:39
distributing code from branch Dev #82
The products are seperated based on the Categories:
Generation and Transmission

Need to work on why the products are not
being published as per categories
@Waldleufer Waldleufer changed the base branch from dev to main June 30, 2021 12:38
@Waldleufer Waldleufer changed the base branch from main to dev June 30, 2021 12:42
Waldleufer and others added 2 commits June 30, 2021 14:48
Distribution of current state
Issue #107.
Sort the projects based on categories and Scenarios.
Created a project Scernario map which can e further used for processing.

Co-authored-by: Mani Anand <mani.anand@fau.de>
Copy link
Collaborator Author

@Waldleufer Waldleufer left a comment

Choose a reason for hiding this comment

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

lgtm! 🙂

ScoutAtlas and others added 21 commits July 1, 2021 10:29
Co-authored-by: Martin Wagner <martin.wagner@fau.de>
Checks if useDummyData is true and depending on that getSimaProducts or getDummyProducts.

Co-authored-by: Martin Wagner <martin.wagner@fau.de>
Co-authored-by: Martin Wagner <martin.wagner@fau.de>
Co-authored-by: Julian Oelhaf <julian.oelhaf@posteo.de>
Co-authored-by: Julian Oelhaf <julian.oelhaf@posteo.de>
Co-authored-by: Julian Oelhaf <julian.oelhaf@posteo.de>
Co-authored-by: Julian Oelhaf <julian.oelhaf@posteo.de>
Co-authored-by: Sai Varun Varanasi <varun.sai.varanasi@fau.de>
Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Co-authored-by: Julian Oelhaf <julian.oelhaf@posteo.de>
Co-authored-by: Martin Wagner <martin.wagner@fau.de>
Co-authored-by: Martin Wagner <martin.wagner@fau.de>
Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Co-authored-by: Sai Varun Varanasi <varun.sai.varanasi@fau.de>
Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Co-authored-by: Julian Oelhaf <julian.oelhaf@posteo.de>
Co-authored-by: Martin Wagner <martin.wagner@fau.de>
Co-authored-by: Sai Varun Varanasi <varun.sai.varanasi@fau.de>
Co-authored-by: Martin Wagner <martin.wagner@fau.de>
"Slice'n dice"

Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Co-authored-by: Martin Dürsch <martin.duersch@t-online.de>
Copy link
Collaborator Author

@Waldleufer Waldleufer left a comment

Choose a reason for hiding this comment

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

finally!

by creation of global constants
to test the newly implemented functionality
and put basline scenarios first
to streamline scenario handling
the data Structure how the projects are currently stored.
... might not be used in the end, I won't be sad if it gets removed.
@MartinDuersch MartinDuersch marked this pull request as ready for review July 2, 2021 12:21
Copy link
Collaborator

@ScoutAtlas ScoutAtlas left a comment

Choose a reason for hiding this comment

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

Very smart and elegant solution.

Copy link
Collaborator

@MartinDuersch MartinDuersch left a comment

Choose a reason for hiding this comment

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

Greatly improved data management

and implementing tweaks as suggested by Reviewers
@Waldleufer Waldleufer merged commit ec244b0 into dev Jul 5, 2021
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.

4 participants