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.

Sign up for GitHub

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

Jump to bottom

Pr/ts methods #9

Closed
wants to merge 33 commits into from
Closed

Pr/ts methods #9

wants to merge 33 commits into from

Conversation

prao7
Copy link
Collaborator

@prao7 prao7 commented Apr 12, 2024

@rodrigomha @sourabhdalvi

Changes made:

  • Time series methods added to portfolio.jl
  • Base 5 bus system added for testing, will be adding wind profile etc. over the weekend too
  • DRAFT: test_time_series.jl added, still messing around with these tests, will finish these up over the weekend
  • Some getter and setter functions added, need checking for storage.jl, especially battery chemistry, as I was thinking of importing a PSY.storage test component potentially here

sourabhdalvi and others added 30 commits March 25, 2024 10:20
@sourabhdalvi
bug fix in struct defination
79d8928
@sourabhdalvi
adding some examples
a6c9880
fixed UUID
b21e855
@sourabhdalvi
Merge branch 'main' into sd/dev
2d553ed
@sourabhdalvi
removing function redefination
fc74eb2
@rodrigomha
add setter methods
825c74c
@rodrigomha
add more base.jl methods for portfolios
f6eb445
@rodrigomha
update removal of components
ba277d7
@rodrigomha
add script example to add technology
3c9283c
@rodrigomha
remove technology script
b03645b
add get_time_series_multiple
032fcf5
adding time series transformation functions
0308472
adding storage time series methods
2a3e68c
adding time series by key get method
dcc9bcd
@sourabhdalvi
bug fix in struct defination
530178c
@sourabhdalvi
adding some examples
3444752
fixed UUID
6e1b2a3
@rodrigomha
add pretty print
695cc6e
@rodrigomha
add pretty tables
4a04b9e
@rodrigomha
add portfolio constructors
6e62671
@sourabhdalvi
adding auto generated structs
71bbb45
@sourabhdalvi
adding serialization
2083663
@sourabhdalvi
updating test script
f8d5409
@sourabhdalvi
adding to_json function
06bd0c8
committing changes prior to rebasing
bf60f2a
fixing bugs on time series data methods
327ebb0
added base 5 bus system for the test
e0f8eda
Merge branch 'sd/dev' into pr/ts_methods
2016ef6
added dates, revising time series methods
30a0d33
changing where timeseriesdata functions are and adding tests
6b3e7db
@rodrigomha rodrigomha changed the base branch from sd/dev to rh/psy_methods April 15, 2024 21:23
@rodrigomha rodrigomha self-requested a review April 15, 2024 21:23
@rodrigomha rodrigomha changed the base branch from rh/psy_methods to rh/dev April 17, 2024 20:21
@jd-lara jd-lara mentioned this pull request May 31, 2024
Merged
@jd-lara
Copy link
Member

jd-lara commented May 31, 2024

closed in favor of #11

@jd-lara jd-lara closed this May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodrigomha rodrigomha Awaiting requested review from rodrigomha

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@prao7 @jd-lara @sourabhdalvi @rodrigomha