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

Add simplified PV system model based on 5p analytical modeling approach #1575

Open
5 tasks done
LauraMaier opened this issue Jan 28, 2022 · 12 comments · May be fixed by #1766
Open
5 tasks done

Add simplified PV system model based on 5p analytical modeling approach #1575

LauraMaier opened this issue Jan 28, 2022 · 12 comments · May be fixed by #1766
Assignees

Comments

@LauraMaier
Copy link
Contributor

LauraMaier commented Jan 28, 2022

The IBPSA library is currently missing a PV system model which can be used in the context of building performance simulations.
The intended model will have the following features:

  • The model will consist of three sub-models, namely a cell temperature, an I-V-characteristic, and a radiation and absorbance model

  • The model's input are the weather bus BoundaryConditions.WeatherData.Bus (DryBulbTemp, wind speed and global horizontal irradiance)

  • The model's output is the DCOutputPower as Real Interface

The sub-models have the following characteristics:

1. Cell temperature model

2. I-V-characteristic

3. Radiation and absorbance model

What steps are necessary to solve this issue?

  • Add package structure Electrical.PVSystems
  • Add model Electrical.PVSystems.PVSystem
  • Implement example of annual simulation
  • Implement model as validation data using NIST measurements
  • Add regression tests and Simulate and Plot scripts
@LauraMaier LauraMaier self-assigned this Jan 28, 2022
@LauraMaier
Copy link
Contributor Author

@mwetter What do you think about the model's features?

@mwetter
Copy link
Contributor

mwetter commented Jan 28, 2022

@LauraMaier : This would be a good addition. I would prefer to put it in IBPSA.BaseClasses.PVSystems because most if not all electrical libraries have some electrical connector (but not all use the same connector). Then, these libraries can extend from your basic model and add in their extended model the appropriate connector. Users can then access the model with the compatible connectors. For example, in Buildings, we would probably have something like

within Buildings.Electrical.DC.Sources.PVSimple;
model PVDetailed
  extends Buildings.Electrical.BaseClasses.PVSystems.YourNewModel;

// Terminal that will use the real output, but in the physical quantities used in the library
Buildings.Electrical.DC.Interfaces.Terminal_p terminal "Terminal for electrical connection";
....

...

@nytschgeusen
Copy link
Contributor

nytschgeusen commented Jan 30, 2022

@LauraMaier: This is a very a good suggestion. As I told you in our discussion during BS 2021 in Bruges we also have in our BuildingSystems library a PV generator model which is subdivided in an electrical, an optical and a thermal model (https://github.com/UdK-VPT/BuildingSystems/tree/master/BuildingSystems/Technologies/Photovoltaics). If we put all together our knowledge in a common PV model for the IBPSA library this would be great.

I think one important topic which we should duchess will be the accuracy of the model (e.g. one diode or two diode model) and the availability of the therefor needed model parameters. For this purpose, it could also help to take a look on the database of the Polysun simulation tool. This database includes thousand of different PV generators from worldwide manufactures and also for generic PV generators.

@LauraMaier
Copy link
Contributor Author

@mwetter Thanks for th suggestion. I will gladly adapt it to the proposed structure

@LauraMaier
Copy link
Contributor Author

@nytschgeusen Thanks, Christoph. Indeed, I have already scanned your very good models and will merge our modeling approach with yours! If you prefer to have a bilateral discussion before I start implementing, I am happy to arrange an online meeting. What do you think?

@nytschgeusen
Copy link
Contributor

@LauraMaier: yes, an online Meeting would be great. Here you can find for the beginning of our discussion a relatively old publication, in which our PV model was applied for solar driven cooling systems: https://www.aivc.org/sites/default/files/p_1460.pdf

LauraMaier added a commit that referenced this issue Jun 14, 2022
LauraMaier added a commit that referenced this issue Jun 14, 2022
LauraMaier added a commit that referenced this issue Oct 7, 2022
LauraMaier added a commit that referenced this issue Oct 7, 2022
@LauraMaier
Copy link
Contributor Author

@Mathadon and @nytschgeusen:

@mwetter and me just discussed about the potential structure of the PV system and inherently also the structure of the new electrical package of the IBPSA library.
As you might remember, we had a discussion on which connectors we would be using in the future as electrical connectors and all of us decided that we would opt for the electrical interfaces as they are currently implemented in the buildings library: https://github.com/lbl-srg/modelica-buildings/blob/master/Buildings/Electrical/Interfaces/
Buildings.Electrical.Interfaces
Following this approach, we would introduce a partialTerminal and a Terminal as connector.
In addition, PhaseSystems would have to be defined as repleceable package.
Basically, by agreeing on using the connectors as implemented in the Buildings library, we would also adopt the whole package structure (AC and DC package, Separation into different phase type models, etc.). As we are not sure, if you are fine with the additional requirements which add up to "we use the electrical connectors of the building library", we wanted to check with you before implementing it.

So what do you think?
Would you favor another solution/structure?

@nytschgeusen
Copy link
Contributor

nytschgeusen commented Oct 11, 2022

@LauraMaier and I just discussed the proposed reuse of the electrical infrastructure models of the Buildings library. We agreed that we will use a subset of them for the new PV system package.

@LauraMaier
Copy link
Contributor Author

Hi @nytschgeusen, @mwetter, and @Mathadon:
I just pushed a first version of the new Electrical package structure following and using many existing models of the Buildings library. I found many of the already implemented models extremely helpful but did not include all of them to reduce redundancy.

Please note that the packages DataBase, BaseClasses and PVSystem still follow an old logic that we first wanted to implement and can be ignored!

As these changes greatly affect the further model development, I would like to get your feedback if you agree with the basic structure and the use of existing smaller models (Interfaces, Sources, Loads, etc.) of the Buildings library.
If so, I can proceed with the integration of the PV models following the implemented structure as described in this issue and also add the conversion scripts of the newly introduced models.

LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 6, 2023
LauraMaier added a commit that referenced this issue Oct 8, 2023
LauraMaier added a commit that referenced this issue Oct 13, 2023
LauraMaier added a commit that referenced this issue Oct 13, 2023
LauraMaier added a commit that referenced this issue Oct 13, 2023
LauraMaier added a commit that referenced this issue Oct 17, 2023
LauraMaier added a commit that referenced this issue Dec 12, 2023
LauraMaier added a commit that referenced this issue Dec 12, 2023
LauraMaier added a commit that referenced this issue Dec 12, 2023
LauraMaier added a commit that referenced this issue Dec 14, 2023
LauraMaier added a commit that referenced this issue Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants