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

Move AixLib.Fluid.Storage.Storage in Obsolete package and rename AixLib.Fluid.Storage.BufferStorage #1171

Open
LauraMaier opened this issue Oct 21, 2021 · 12 comments · May be fixed by #1226
Open

Comments

@LauraMaier
Copy link
Contributor

The model AixLib.Fluid.Storage.Storage is a simplified buffer storage model, which is used only rarely.
Additionally, a second buffer storage model exists in AixLib.Fluid.Storage.BufferStorage, which is more detailed and validated.
This is why, I propose to move the simplified model in the Obsolete package.
Plus, the model AixLib.Fluid.Storage.BufferStorage can also be used as e.g. a domestic hot water storage, which is why we propose refactoring it to AixLib.Fluid.Storage.ThermalEnergyStorage.

@LauraMaier LauraMaier added this to the Hackday 10.2021 milestone Oct 21, 2021
@LauraMaier LauraMaier self-assigned this Oct 21, 2021
@LauraMaier
Copy link
Contributor Author

LauraMaier commented Jan 4, 2022

Update:
AixLib.Fluid.Storage.Storage will not be deleted because the simple storage might still be needed for broad parameter studies (i.e. varying the storage volume).

Therefore, the issue comprises the following tasks:

  • Rename AixLib.Fluid.Storage.BufferStorage to AixLib.Fluid.Storage.DetailedStorage
  • Rename AixLib.Fluid.Storage.Storage to AixLib.Fluid.Storage.SimpleStorage
  • Update Documentation to clarify differences between storage models and purposes
  • Spring cleaning in AixLib.Fluid.Storage.Storage

@FWuellhorst
Copy link
Contributor

Alternative:
Check which integers depend on the volume and make them not final anymore.

final parameter Integer nHC1Up=integer(ceil(data.hHC1Up/(data.hTank/n)));
 final parameter Integer nHC1Low=integer(floor(data.hHC1Low/(data.hTank/n))+1);
 final parameter Integer disHC1 = nHC1Up-nHC1Low+1;

 final parameter Integer nHC2Up=integer(ceil(data.hHC2Up/(data.hTank/n)));
 final parameter Integer nHC2Low=integer(floor(data.hHC2Low/(data.hTank/n))+1);
 final parameter Integer disHC2 = nHC2Up-nHC2Low+1;

 final parameter Integer nHR=integer(AixLib.Utilities.Math.Functions.round(data.hHR/(data.hTank/n) + 0.5,0));

 final parameter Integer nTS1=integer(AixLib.Utilities.Math.Functions.round(data.hTS1/(data.hTank/n) + 0.5,0));
 final parameter Integer nTS2=integer(AixLib.Utilities.Math.Functions.round(data.hTS2/(data.hTank/n) + 0.5,0));

 final parameter Integer nLowerPortDemand=integer(max(AixLib.Utilities.Math.Functions.round(data.hLowerPortDemand/(data.hTank/n) + 0.5,0),1));
 final parameter Integer nUpperPortDemand=integer(min(AixLib.Utilities.Math.Functions.round(data.hUpperPortDemand/(data.hTank/n) + 0.5,0),n));
 final parameter Boolean inpLowLayDemand=(nLowerPortDemand == 1); //if there is an input at the lowest layer
 final parameter Boolean inpHigLayDemand=(nUpperPortDemand == n);

 final parameter Integer nLowerPortSupply=integer(max(AixLib.Utilities.Math.Functions.round(data.hLowerPortSupply/(data.hTank/n) + 0.5,0),1));
 final parameter Integer nUpperPortSupply=integer(min(AixLib.Utilities.Math.Functions.round(data.hUpperPortSupply/(data.hTank/n) + 0.5,0),n));
 final parameter Boolean inpLowLaySupply=(nLowerPortSupply == 1); //if there is an input at the lowest layer
 final parameter Boolean inpHigLaySupply=(nUpperPortSupply == n);

 final parameter Integer[n] inpActLay = cat(1,{if (nLowerPortDemand==k or nLowerPortSupply==k) then (if (nUpperPortDemand==k or nUpperPortSupply==k) then 2 else 1) else (if (nUpperPortDemand==k or nUpperPortSupply==k) then 1 else 0) for k in 1:n});
 final parameter Integer[n] portsLayer=cat(1,{if inpActLay[1]==1 then 3 else 1},{inpActLay[k]*2+2 for k in 2:n-1},{if inpActLay[n]==1 then 3 else 1});

@FWuellhorst FWuellhorst added this to To do in Hackday_1_2022 Jan 26, 2022
LauraMaier added a commit that referenced this issue Jan 27, 2022
LauraMaier added a commit that referenced this issue Jan 27, 2022
@FWuellhorst FWuellhorst moved this from To do to Doing in Hackday_1_2022 Jan 27, 2022
@LauraMaier LauraMaier linked a pull request Jan 28, 2022 that will close this issue
@LauraMaier LauraMaier linked a pull request Jan 28, 2022 that will close this issue
LauraMaier added a commit that referenced this issue Mar 23, 2022
@ebc-aixlib-bot
Copy link
Contributor

The branch 1171 has been inactive for more than 155 days. The branch is automatically deleted after 180 days. If you want to keep the branch, add changes to the branch. A pull request is created and the branch is then deleted. If you want to restore the branch, go to the closed pull requests and restore your branch.
User name: Laura Maier
Branch URL: https://github.com/RWTH-EBC/AixLib/tree/1171

@LauraMaier
Copy link
Contributor Author

I did not put the simple storage in the Obsolete package as it is still needed for parameter studies, etc.

@ebc-aixlib-bot
Copy link
Contributor

The branch 1171 has been inactive for more than 186 days. Cannot create a pull request, because there are no commits between development and 1171
User name: Laura Maier

@FWuellhorst
Copy link
Contributor

@DaJansenGit : Did the bot just close the issue and the branch?

@DaJansenGit
Copy link
Member

DaJansenGit commented Aug 2, 2022

@FWuellhorst: The issue was closed, but the branch not. The problem is that the CI associated this issue with this branch which was created by the already deactivated GitHub action which we found not convenient. But the pushes made by @LauraMaier are against this branch .

If you are fine with it, I would just delete the automatic created branch, reopen this issue and go on with the work in this branch . :)

@ebc-aixlib-bot
Copy link
Contributor

The branch Issue1171_Move_AixLib_Fluid_Storage_Storage_in_Obsolete_package_and_rename_AixLib_Fluid_Storage_BufferStorage has been inactive for more than 110 days. The branch is automatically deleted after 180 days. If you want to keep the branch, add changes to the branch. A pull request is created and the branch is then deleted. If you want to restore the branch, go to the closed pull requests and restore your branch.
User name: larissakuehn
Branch URL: https://github.com/RWTH-EBC/AixLib/tree/Issue1171_Move_AixLib_Fluid_Storage_Storage_in_Obsolete_package_and_rename_AixLib_Fluid_Storage_BufferStorage

@ebc-aixlib-bot
Copy link
Contributor

The branch Issue1171_Move_AixLib_Fluid_Storage_Storage_in_Obsolete_package_and_rename_AixLib_Fluid_Storage_BufferStorage has been inactive for more than 141 days. The branch is automatically deleted after 180 days. If you want to keep the branch, add changes to the branch. A pull request is created and the branch is then deleted. If you want to restore the branch, go to the closed pull requests and restore your branch.
User name: larissakuehn
Branch URL: https://github.com/RWTH-EBC/AixLib/tree/Issue1171_Move_AixLib_Fluid_Storage_Storage_in_Obsolete_package_and_rename_AixLib_Fluid_Storage_BufferStorage

@larissakuehn larissakuehn removed their assignment Dec 6, 2022
@FelixStege
Copy link
Contributor

@LauraMaier will this be tackled on the upcoming hackday or can we remove the label?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

7 participants