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

Dev #749

Merged
merged 72 commits into from
Feb 18, 2022
Merged

Dev #749

merged 72 commits into from
Feb 18, 2022

Conversation

dave-doty
Copy link
Member

No description provided.

cesarg707 and others added 30 commits December 8, 2021 20:09
…and_5p to be consistent with naming convention in ligate reducer (and be more unambiguous about what it means)
… match convention, and changed convention of order of strand parameters to `join_two_strands_with_substrands`
…well as added naming behavior for ligate/crossovers

added behavior where names from a strand that has been nicked, has had a deleted crossover, or has had a deleted domain are inherited by the new strands. Also fixed naming when ligating two strands or adding a crossover strand between strands
…coding-warning-on-startup-#697

Fixes #697; fix JSON encoding warning on startup
…ifferent sequence to bound strand" checkbox when "assign complement to bound strands" is unchecked
…-warn-if-assigning-different-sequence-to-bound-strand-checkbox-when-assign-complement-to-bound-strands-is-unchecked

closes #709; in DNA sequence assignment, disable "warn if assigning different sequence to bound strand" checkbox when "assign complement to bound strands" is unchecked
…er-previously-chosen-M13-rotation-in-Assign-DNA-sequence-dialog

closes #711: remember previously chosen M13 rotation in Assign DNA sequence dialog
…tate (#703)

* Add and use svg_position to DesignMainHelicesProps

* Remove usage of reassign_svg_position in reducers and Design constructor

* Committing to save work in this branch

* Remove usage of Helix.svg_position

* Add back helix_group_change_reducer

* Add some missing props

* Add more missing props

* Fix invert xy bug

* Remove unused helix update

* removed redundant toMap step before calling util.helices_assign_svg by changing parameter type of helices in that function

* removed creation of redundant list in `helices_assign_svg`

* changed all references to `invert_xy` or `invert_yz` to `invert_y`

* fixed bug in calculating SVG positions when displaying only selected helices

* fixed erroneous reference to DNAFileType

* Update util.dart

* add comments to helix_idx_to_svg_position_map memoized getter explaining the design decision

* Remove Helix.invert_y

See #703 (comment)

* Add unit test for #703 (comment)

* Set missing invert_y prop in ConnectedDesignMainArrows

* Remove unneeded data structures and sorting in helices_assign_svg

* commented out StopWatch print statements in helix_idx_to_svg_position_map getter

Co-authored-by: David Doty <doty@ucdavis.edu>
…m multiple helix groups are selected (#720)

* Fixes #668; fix crash error when deselecting strand while strands from multiple helix groups are selected

* Fix bug, forgot to account for negative changes to idx/offset

* Print out unmatched strand to facilitate debugging
…domains-should-be-reselected-after-move

#722 moving domain should be disallowed if overlapping another domain
…e-drawing-of-strands-when-connecting-a-crossover

fixes #717: Disable drawing of strands when connecting a crossover
dave-doty and others added 29 commits January 19, 2022 16:50
…aste-strands-breaks-when-helix-counts/indices-are-not-sufficient

#732 fix bug where copy/paste strands breaks when helix counts/indices are not sufficient in destination design
* Update dependencies and Dart SDK in workflows

* Fix template and remove deprecated function

* Add pubspec.lock

* Set minimum SDK version to 2.11

* Fix syntax error in workflow file

* Name checkout actions to improve readability
…et domain on any helix, and fixed crash bug in setting this for helices with no domains
…-offset-uniform

added option to set max offset of every helices based on largest-offset domain on any helix, and fixed crash bug in setting this for helices with no domains
* Create pull_request_template.md

* Indicate to fill in automatic template in CONTRIBUTING.md
…-length-of-modification-connector-

fixes #743: change length of modification "connector"
…e-display-of-helices-in-main-view

closes #746: disable display of helices in main view
…g-strands-should-keep-at-least-one-existing-name

#681 merging strands should keep at least one existing name
…on prompt for the user if they click "Reset local settings"
…ocal-settings-button-#719

closes #719: add "reset local settings" button
@dave-doty dave-doty merged commit c5dbfc0 into main Feb 18, 2022
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.

5 participants