-
Notifications
You must be signed in to change notification settings - Fork 2
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
RT59314 #595
base: main
Are you sure you want to change the base?
RT59314 #595
Conversation
zaucker
commented
Jan 31, 2025
- Standard-Werte für Schweinefütterung eingerichtet
- Race-condition eliminiert, die ev dazu geführt hat, dass nach Input-Änderungen nicht neu gerechnet wurde (unklar, da nicht gut reproduzierbar)
@ChHaeni bitte die Änderungen an dem .nhd Files anschauen und verifizieren, dass die Defaults wie gewünscht funktionieren. |
@@ -517,10 +530,14 @@ Richner, W., Flisch, R., Mayer, J., Schlegel, P., Zähner, M., Menzi, H., 2017. | |||
Walther et al. (2001). | |||
?if Kantonal_LU | |||
++formula | |||
my $energyContent = lc In(energy_content) eq 'standard' ? $TE->{'standard_energy_content_' . Out(animalcategory)} : In(energy_content); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wäre wohl etwas schöner eine Output Variable "energy_content" einzuführen, wenn auch nicht notwendig... Gleiches für "energy_content" und "crude_protein" bei Pig.
@zaucker Entschuldigung, ich war etwas zu schnell mit approving. Muss noch verstehen woher die neuen "default" Werte kommen & wieso die sich zu den "standard" Werten unterscheiden sollen... |
Das hatte ich so von Thomas verstanden. Das eine sind Standardwerte für die Berechnung und die neuen Defaults entsprechen den Angaben im Hilfetext und sind typische Werte für die Schweiz.Aber bitte mit Thomas verifizieren.Am 03.02.2025 um 10:48 schrieb Christoph Häni ***@***.***>:
@zaucker Entschuldigung, ich war etwas zu schnell mit approving. Muss noch verstehen woher die neuen "default" Werte kommen & wieso die sich zu den "standard" Werten unterscheiden sollen...
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Hab das mit Thomas besprochen & verstehe jetzt die Vorgehensweise - war/ist einfach etwas verwirrend mit standard vs. default... |
Das ging mir ähnlich. Ev muss ich nachbringen Kommentar ergänzen.Hast du auch getestet, ob die Testinstanz wie gewünscht funktioniert?Gruss aus den SkiferienFritzAm 03.02.2025 um 14:53 schrieb Christoph Häni ***@***.***>:
Das hatte ich so von Thomas verstanden. Das eine sind Standardwerte für die Berechnung und die neuen Defaults entsprechen den Angaben im Hilfetext und sind typische Werte für die Schweiz.Aber bitte mit Thomas verifizieren.Am 03.02.2025 um 10:48 schrieb Christoph Häni @.>: @zaucker Entschuldigung, ich war etwas zu schnell mit approving. Muss noch verstehen woher die neuen "default" Werte kommen & wieso die sich zu den "standard" Werten unterscheiden sollen... —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.>
Hab das mit Thomas besprochen & verstehe jetzt die Vorgehensweise - war/ist einfach etwas verwirrend mit standard vs. default...
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|