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.

Sign up for GitHub

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

Jump to bottom

Cvs n rows issue #337 #51

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Cvs n rows issue #337 #51

wants to merge 9 commits into from

Commits on May 22, 2024

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

    The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JAVA-COMAMAZONREDSHIFT-6841702

Co-authored-by: snyk-bot <[email protected]>
    @janblom @snyk-bot
    janblom and snyk-bot authored May 22, 2024
    Configuration menu
    Copy the full SHA
    1dab445 View commit details
    Browse the repository at this point in the history

Commits on May 23, 2024

  1. Update README.md 

    Add statement about intended use.
    @janblom
    janblom authored May 23, 2024
    Configuration menu
    Copy the full SHA
    c4da141 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2024

  1. 'n rows' for csv files now get correctly populated with the number of… 

    … lines read from the csv file
    @EIjo
    EIjo committed Jul 17, 2024
    Configuration menu
    Copy the full SHA
    6e79f1f View commit details
    Browse the repository at this point in the history

  2. decremented n_row value by one, as header was counted as well in prev… 

    …ious implementation
    @EIjo
    EIjo committed Jul 17, 2024
    Configuration menu
    Copy the full SHA
    f195003 View commit details
    Browse the repository at this point in the history

  3. updated ScanReport-reference-v0.10.7-csv.xlsx to include n_row values… 

    … instead of -1
    @EIjo
    EIjo committed Jul 17, 2024
    Configuration menu
    Copy the full SHA
    f4bcc4b View commit details
    Browse the repository at this point in the history

Commits on Jul 18, 2024

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

    @EIjo
    EIjo committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    f8bdc38 View commit details
    Browse the repository at this point in the history

Commits on Aug 8, 2024

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

    @EIjo
    EIjo committed Aug 8, 2024
    Configuration menu
    Copy the full SHA
    1698f6e View commit details
    Browse the repository at this point in the history

Commits on Aug 12, 2024

  1. Added feature to scan csv files in splits.

    @EIjo
    EIjo committed Aug 12, 2024
    Configuration menu
    Copy the full SHA
    1408e13 View commit details
    Browse the repository at this point in the history

Commits on Aug 13, 2024

  1. Reverted scanReport as well as changed rowCount to not show ~ to indi… 

    …cate an estimate (some other manner of comunicating estimates should be used)
    @EIjo
    EIjo committed Aug 13, 2024
    Configuration menu
    Copy the full SHA
    51fa0bb View commit details
    Browse the repository at this point in the history