Merge pull request #429 from StoXProject/develop

arnejohannesholmin · web-flow · commit 70f694aea419 · 2025-03-27T10:11:57.000+01:00
Develop
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
