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

Refactor the code and move some functions into class methods #1

Closed
wants to merge 31 commits into from

Conversation

Bachibouzouk
Copy link

@Bachibouzouk Bachibouzouk commented Mar 28, 2022

Closes #2

@dhungelgd I will move your changes into this PR as I go over the review of #4

This should constitute the next PR on the original RAMP repository, as soon as RAMP-project#34 is merged

@Bachibouzouk Bachibouzouk requested a review from dhungelgd March 28, 2022 08:51
@Bachibouzouk Bachibouzouk force-pushed the feature/refactor-code branch from 042bef3 to eb47df5 Compare May 18, 2022 10:58
@Bachibouzouk Bachibouzouk changed the base branch from master to feature/import-export May 18, 2022 10:58
@Bachibouzouk Bachibouzouk changed the title WIP Start refactoring the code and move some functions into class methods May 18, 2022
@Bachibouzouk Bachibouzouk force-pushed the feature/refactor-code branch from 41f39c9 to 79b72ab Compare May 19, 2022 22:46
@Bachibouzouk Bachibouzouk changed the title Start refactoring the code and move some functions into class methods Refactor the code and move some functions into class methods May 20, 2022
@Bachibouzouk Bachibouzouk force-pushed the feature/refactor-code branch 4 times, most recently from add32f0 to 5353911 Compare June 3, 2022 22:49
@Bachibouzouk Bachibouzouk force-pushed the feature/import-export branch 2 times, most recently from a98e3b5 to 99abb79 Compare June 8, 2022 20:19
@Bachibouzouk Bachibouzouk force-pushed the feature/refactor-code branch 2 times, most recently from e1bf722 to e428d1e Compare July 6, 2022 14:05
@Bachibouzouk Bachibouzouk force-pushed the feature/refactor-code branch from e428d1e to c154922 Compare July 6, 2022 14:28
FLomb and others added 12 commits July 8, 2022 15:37
…port

Use a spreadsheet as input file as an alternative to .py files
- Mapping between old and new parameters in file constants.py
- Move the class Appliance outside of User class
Provide two ways to define power timeseries
Move the peak time frame calculation within a function
to allow for unittesting.

Use maximum_profile instead of the misleading window_curve. A User
instance maximum_profile constists of the aggregation of its Appliance
instances maximum_profiles

Co-authored-by: Gokarna.Dhungel <[email protected]>
Identifies a random switch on time within the available functioning
windows

Co-authored-by: Gokarna.Dhungel <[email protected]>
Bachibouzouk and others added 19 commits July 29, 2022 14:15
Also assign the random cycles to an attribute of the appliance

Co-authored-by: Gokarna.Dhungel <[email protected]>
Also explicitely assign attributes in __init__ method of Appliance class

Co-authored-by: Gokarna.Dhungel <[email protected]>
Co-authored-by: Gokarna.Dhungel <[email protected]>
Stress the relations between variables and equations of the seminal
paper and the code with comments

Co-authored-by: Gokarna.Dhungel <[email protected]>
The method is called generate_profile and update the daily_use attribute

Co-authored-by: Gokarna.Dhungel <[email protected]>
Enhance code readability and decrease risk of errors by typos

Co-authored-by: Gokarna.Dhungel <[email protected]>
Add further quotes to the seminal paper in the docstrings
Move the code which determines whether the indexes are within or out of
the peak window outside the function, so it makes the argument of the
method more meaningful
One to define a duty_cycle and the otherone to choose between two duty
cycles
@Bachibouzouk Bachibouzouk force-pushed the feature/refactor-code branch from c154922 to 99175f1 Compare July 29, 2022 12:15
@Bachibouzouk
Copy link
Author

This PR is now replaced by RAMP-project#36

Bachibouzouk pushed a commit that referenced this pull request Sep 28, 2022
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.

Next steps
3 participants