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

RT59314 #595

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

RT59314 #595

wants to merge 10 commits into from

Conversation

zaucker
Copy link
Contributor

@zaucker 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)

@zaucker
Copy link
Contributor Author

zaucker commented Jan 31, 2025

@ChHaeni bitte die Änderungen an dem .nhd Files anschauen und verifizieren, dass die Defaults wie gewünscht funktionieren.

@zaucker zaucker requested a review from ChHaeni January 31, 2025 10:22
@@ -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);
Copy link
Collaborator

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.

@ChHaeni ChHaeni self-requested a review February 3, 2025 09:34
@ChHaeni
Copy link
Collaborator

ChHaeni commented Feb 3, 2025

@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...

@zaucker
Copy link
Contributor Author

zaucker commented Feb 3, 2025 via email

@ChHaeni
Copy link
Collaborator

ChHaeni commented Feb 3, 2025

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...

@zaucker
Copy link
Contributor Author

zaucker commented Feb 3, 2025 via email

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.

2 participants