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

Topic Modeling: save selected topic as schema_only #478

Merged
merged 2 commits into from
Dec 4, 2019

Conversation

ajdapretnar
Copy link
Collaborator

@ajdapretnar ajdapretnar commented Dec 2, 2019

Issue

Fixes #213.

Description of changes

Save a selection and load it upon reloading a workflow.

TODO: solve the issue of tests.

Includes
  • Code changes
  • Tests
  • Documentation

@ajdapretnar
Copy link
Collaborator Author

@VesnaT Please help with tests. I have tried all morning to come up with something sensible, but I can't figure out how to pass the selection around.

@ajdapretnar ajdapretnar changed the title Topic Modeling: save selected topic as schema_only [WIP] Topic Modeling: save selected topic as schema_only Dec 2, 2019
@VesnaT
Copy link
Contributor

VesnaT commented Dec 2, 2019

Check ajdapretnar#1

@codecov-io
Copy link

codecov-io commented Dec 3, 2019

Codecov Report

Merging #478 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #478      +/-   ##
=========================================
+ Coverage    85.6%   85.7%   +0.09%     
=========================================
  Files          34      34              
  Lines        2001    2001              
  Branches      336     336              
=========================================
+ Hits         1713    1715       +2     
+ Misses        248     247       -1     
+ Partials       40      39       -1

@ajdapretnar
Copy link
Collaborator Author

ajdapretnar commented Dec 4, 2019

I fixed tests because assert_allclose could not be called on dtype object. And the two outputs had to be re-sorted due to gensim using 'quicksort' rather than 'mergesort'.

Anyhow, test now pass locally, but Travis is still complaining the second output is None, despite the processEvent. 😩 I am throwing this widget out of the window the next change I get...

@ajdapretnar ajdapretnar changed the title [WIP] Topic Modeling: save selected topic as schema_only Topic Modeling: save selected topic as schema_only Dec 4, 2019
@ajdapretnar ajdapretnar merged commit 3f01f69 into biolab:master Dec 4, 2019
@ajdapretnar ajdapretnar deleted the topic-settings branch January 21, 2021 12:55
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.

Topic Modeling: retain settings
3 participants