-
Notifications
You must be signed in to change notification settings - Fork 1
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
Various fixes & updates #37
Conversation
Change default height of tree
Tree page bugs
trees page quick fix
fix: uncommented tests
Warning message when no samples selected
More iGNN fixes
Ignn page: warnings
Very nice work! Thank you 👍 The way some parameters are set by default, in datastore.py, are causing the stats to fail by default. The one-way stat should work as soon as one sample set is chosen, however, the default is expecting the sample sets : [0, 1], so it cannot load them if you put any other configuration in the Individuals tabs. Same for the multi-way stat, but it's for the same reason. By default it's expecting the sample set [0,1,2] and indexes [(0,1), (0,2), (1,2)]. Some ideas : When selecting individuals, sort the list so that the selected samples show up first. I did not noticed the second page and was looking for the orangutan at first, when selecting it by its sample set (5). In the trees tab, it would be possible to initialize the first position to the first tree in the tree sequence, by using tree_sequence.first().interval.left or something like that, I guess? |
@tforest we have created issues to address your comments. To solve the issue with the statistics page, we will set all samples as selected by default (see issue 67). Further improvements to the statistics page are coming, see PR 63. We added an issue to fix the sorting of individuals, see issue 66. This won't be fixed in this PR, but possible in a future one. After discussions with Per, we think that the reason why there is no tree at index 0 is due to the specific test data. This should not be a problem for other datasets. We added an issue for this (issue 68) but labeled it as wontfix. We will solve a few small issues on the Individual's page before this PR is ready to be merged (67, 69 and 70). |
See the individual closed PRs on our fork: here
Each PR is linked to closed issues we have identified, the issues are named based on the page
Sample sets page:
Individuals page:
Structure page:
iGNN page:
Statistics page:
Trees page:
General: