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

Randomisations - add tree perturbation option #389

Closed
GoogleCodeExporter opened this issue Mar 27, 2015 · 21 comments
Closed

Randomisations - add tree perturbation option #389

GoogleCodeExporter opened this issue Mar 27, 2015 · 21 comments

Comments

@GoogleCodeExporter
Copy link

The randomisations currently work only on the basedata (although see recent 
work on group properties - issue #388).  

A useful addition would be to randomly perturb the positions of nodes on any 
trees used as analysis arguments.  

In the first instance this would work only on the terminals, randomly 
reassigning terminal nodes across the tree while maintaining the topology and 
branch structure.  

Further variants could randomly re-assign any named nodes (for cases where 
internals are named).  Later work can also randomly reassign branch lengths.  

Note:  Need to add references for this sort of work. as it has been implemented 
in other systems.

Original issue reported on code.google.com by shawnlaffan on 13 Jul 2013 at 3:56

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1638.

Add branch for issue related changes

Original comment by shawnlaffan on 13 Jul 2013 at 4:10

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1639.

Implement terminal node shuffle.  Includes initial test.

Original comment by shawnlaffan on 13 Jul 2013 at 5:12

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1640.

Add tests for shuffling (in progress).

Original comment by shawnlaffan on 13 Jul 2013 at 9:07

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1641.

More work on the test for shuffling terminals.  Still too much hard coding, but 
works for now.  

Original comment by shawnlaffan on 13 Jul 2013 at 10:06

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1642.

Start caching the shuffled trees.  Make the methods more generic - more needed 
on that front. 
Add tests to check we use the cached objects, and also that we don't corrupt 
the original args. 

Original comment by shawnlaffan on 14 Jul 2013 at 3:31

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1643.

Test that the shuffled trees are different between randomisation iterations.  

Original comment by shawnlaffan on 14 Jul 2013 at 3:55

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1644.

Fix incorrect search for arguments hash.

Original comment by shawnlaffan on 14 Jul 2013 at 4:09

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1645.

Add a no change shuffle option.  Modify tests to handle new shuffles as they 
are developed.  

Original comment by shawnlaffan on 14 Jul 2013 at 6:54

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1646.

Now testing a second tree used as an analysis argument.  

Original comment by shawnlaffan on 14 Jul 2013 at 11:18

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1647.

Add infrastructure to the GUI.  

Original comment by shawnlaffan on 14 Jul 2013 at 1:30

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Original comment by shawnlaffan on 14 Jul 2013 at 11:55

  • Changed state: Started
  • Added labels: Milestone-Release0.18
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1649.

Merge changes into the trunk.

Original comment by shawnlaffan on 15 Jul 2013 at 12:25

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1650.

Allow abbreviated forms for the shuffling (i.e. we can skip the leading 
"shuffle_").

Original comment by shawnlaffan on 15 Jul 2013 at 1:50

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1651.

Merge latest changes into the trunk.

Original comment by shawnlaffan on 15 Jul 2013 at 1:51

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Default value is coming across as text instead of an array index.  

Argument "no_change" isn't numeric in subroutine entry at 
Biodiverse/GUI/ParametersTable.pm line 177.
Argument "no_change" isn't numeric in subroutine entry at 
Biodiverse/GUI/ParametersTable.pm line 177.

Original comment by shawnlaffan on 15 Jul 2013 at 3:16

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1655.

Avoid an undef warning.  

Original comment by shawnlaffan on 15 Jul 2013 at 4:27

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1656.

Fix default parameter handling for choice types.

Original comment by shawnlaffan on 15 Jul 2013 at 4:46

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1657.

Merge changes to trunk

Original comment by shawnlaffan on 15 Jul 2013 at 5:02

  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

Original comment by shawnlaffan on 16 Jul 2013 at 11:05

@GoogleCodeExporter
Copy link
Author

Mark as fixed
Blocker issue #391 is now fixed.  

Original comment by shawnlaffan on 18 Aug 2013 at 11:09

  • Changed state: Fixed
  • Added labels: ****
  • Removed labels: ****

@GoogleCodeExporter
Copy link
Author

This issue was updated by revision r1768.

Delete branch associated with this issue.  

Original comment by shawnlaffan on 19 Aug 2013 at 10:27

  • Added labels: ****
  • Removed labels: ****

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants