Code

Cvs n rows issue #337#51

Code

Open

EIjo wants to merge 9 commits intomasterthehyve/OHDSI-WhiteRabbit:masterfrom cvs-n-rowsthehyve/OHDSI-WhiteRabbit:cvs-n-rowsCopy head branch name to clipboard

+1,371-1,179

Commits

Commits on May 22, 2024

• fix: rabbit-core/pom.xml to reduce vulnerabilities (OHDSI#413)

  Show description for 1dab445
  

  janblom

  and

  snyk-bot

  authored
  1dab445

Commits on May 23, 2024

• Update README.md

  Show description for c4da141
  janblom

  authored
  c4da141

Commits on Jul 17, 2024

• 'n rows' for csv files now get correctly populated with the number of lines read from the csv file

  EIjo

  committed
  6e79f1f

• decremented n_row value by one, as header was counted as well in previous implementation

  EIjo

  committed
  f195003

• updated ScanReport-reference-v0.10.7-csv.xlsx to include n_row values instead of -1

  EIjo

  committed
  f4bcc4b

Commits on Jul 18, 2024

• Merge remote-tracking branch 'thehyve-wr/master' into cvs-n-rows

  EIjo

  committed
  f8bdc38

Commits on Aug 8, 2024

• Updated rowCount to show an estimate if not all rows are read

  EIjo

  committed
  1698f6e

Commits on Aug 12, 2024

• Added feature to scan csv files in splits.

  EIjo

  committed
  1408e13

Commits on Aug 13, 2024

• Reverted scanReport as well as changed rowCount to not show ~ to indicate an estimate (some other manner of comunicating estimates should be used)

  EIjo

  committed
  51fa0bb