Merge pull request #428 from StoXProject/StoXv4.1.0

arnejohannesholmin · web-flow · commit 06ce9afe84c7 · 2025-03-27T09:34:33.000+01:00
Corrected Sex in CA from NA to a value.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: RstoxData
-Version: 2.1.4-9002
-Date: 2025-03-26
+Version: 2.1.4-9003
+Date: 2025-03-27
 Title: Tools to Read and Manipulate Fisheries Data
 Authors@R: c(
   person(given = "Edvin", 
diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,7 @@
+# RstoxData v2.1.4-9003  (2025-03-27)
+* Corrected error in ICESDatras() where finalHL was not data.table copied from mergedHL, causing Sex to be NA in the CA table.
+
+
 # RstoxData v2.1.4-9002  (2025-03-26)
 * Corrected error in ICESDatras() where HLNoAtLngt was not properly set (Sex set to NA after HLNoAtLngt was calculated).
 
diff --git a/R/StoxExport.R b/R/StoxExport.R
@@ -1233,7 +1233,7 @@ ICESDatrasOne <- function(
 	
 	
 	# To fix the problem that the procedure at the IMR is to not split sex into different subsamples, we set the sex to NA here, so that aggregation to produce noMeas and totalNo sum over sexes. The reason for this is that catCatchWgt and subWeight are in practice summed over sexes, and we do not want to do estimation to split these by sex.
-	finalHL <- mergedHL
+	finalHL <- data.table::copy(mergedHL)
 	finalHL[, sex := NA]
 	
 	# Aggregate hlNoAtLngth and lsCountTot
