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

[ENH] List files that are not loaded #560

Merged
merged 2 commits into from
Sep 4, 2020

Conversation

PrimozGodec
Copy link
Collaborator

@PrimozGodec PrimozGodec commented Aug 19, 2020

Issue

Fixes #551

Description of changes
  • List files that are not loaded
  • Code refactoring
  • Unittests
Includes
  • Code changes
  • Tests

@PrimozGodec PrimozGodec changed the title List files that are not loaded [ENH] List files that are not loaded Aug 19, 2020
@codecov-commenter
Copy link

codecov-commenter commented Aug 19, 2020

Codecov Report

Merging #560 into master will increase coverage by 4.97%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##           master     #560      +/-   ##
==========================================
+ Coverage   68.72%   73.69%   +4.97%     
==========================================
  Files          66       66              
  Lines        7443     7462      +19     
  Branches      996      999       +3     
==========================================
+ Hits         5115     5499     +384     
+ Misses       2157     1751     -406     
- Partials      171      212      +41     

@PrimozGodec PrimozGodec changed the title [ENH] List files that are not loaded [WIP][ENH] List files that are not loaded Aug 19, 2020
@PrimozGodec PrimozGodec changed the title [WIP][ENH] List files that are not loaded [ENH] List files that are not loaded Aug 20, 2020
@ajdapretnar
Copy link
Collaborator

Thanks for working on this! I had something else in mind. And additional output that would list the unread files in a data table. Do you think that would be an overkill? I just think there are several files with long names, a tooltip could get useless quite fast. With a data table, one can even save the output for reference. I dunno. Perhaps this is redundant. What do you think?

@PrimozGodec
Copy link
Collaborator Author

PrimozGodec commented Aug 31, 2020

I think it is a better solution. I added the output with skipped documents.

@ajdapretnar
Copy link
Collaborator

I can't believe Import Documents had no tests before. 😱

@ajdapretnar ajdapretnar merged commit 4207270 into biolab:master Sep 4, 2020
@PrimozGodec PrimozGodec deleted the list-nor-read branch March 29, 2023 10:37
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.

Import Documents: output list of files that couldn't be read
3 participants