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

Pin Biopython 1.81 #200

Merged
merged 1 commit into from
Dec 22, 2023
Merged

Pin Biopython 1.81 #200

merged 1 commit into from
Dec 22, 2023

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Dec 22, 2023

Description of proposed changes

Issue linked in changes. Pin to an exact version which is known to be compatible.

Related issue(s)

Checklist

  • Checks pass
  • Add temporary pin to Biopython 1.82 to show where CI would fail: 330cdb3 (CI)

Issue linked in changes. Pin to an exact version which is known to be
compatible.
@victorlin victorlin self-assigned this Dec 22, 2023
@victorlin victorlin marked this pull request as ready for review December 22, 2023 21:38
@victorlin
Copy link
Member Author

victorlin commented Dec 22, 2023

Merging this so that future builds of the image don't expose effects of nextstrain/augur#1373, which are made apparent by the failing CI when testing without this change:

rsv
        python scripts/cut.py             --oldalignment results/b/G/sequences.aligned.fasta             --slicedalignment results/b/G/G_slicedalignment.fasta             --reference config/breference.gbk             --gene G
        
Traceback (most recent call last):
G
4645 5578 933
  File "/nextstrain/build/scripts/cut.py", line 46, in <module>
    cut(args.oldalignment, args.slicedalignment, args.reference, args.gene, args.min_length)
  File "/nextstrain/build/scripts/cut.py", line 24, in cut
    if len(newrecord.seq.ungap('-')) >= min_length:
AttributeError: 'Seq' object has no attribute 'ungap'
[Fri Dec 22 21:15:53 2023]
Error in rule cut:
seasonal-flu
Traceback (most recent call last):
  File "/nextstrain/augur/augur/__init__.py", line 66, in run
    return args.__command__.run(args)
  File "/nextstrain/augur/augur/ancestral.py", line 305, in run
    features = load_features(args.annotation, args.genes)
  File "/nextstrain/augur/augur/utils.py", line 159, in load_features
    for rec in GFF.parse(in_handle, limit_info=limit_info):
  File "/usr/local/lib/python3.10/site-packages/BCBio/GFF/GFFParser.py", line 793, in parse
    for rec in parser.parse_in_parts(gff_files, base_dict, limit_info,
  File "/usr/local/lib/python3.10/site-packages/BCBio/GFF/GFFParser.py", line 337, in parse_in_parts
    cur_dict = self._results_to_features(cur_dict, results)
  File "/usr/local/lib/python3.10/site-packages/BCBio/GFF/GFFParser.py", line 375, in _results_to_features
    (_, base) = self._add_toplevel_feature(base, feature)
  File "/usr/local/lib/python3.10/site-packages/BCBio/GFF/GFFParser.py", line 582, in _add_toplevel_feature
    new_feature = self._get_feature(feature_dict)
  File "/usr/local/lib/python3.10/site-packages/BCBio/GFF/GFFParser.py", line 591, in _get_feature
    new_feature = SeqFeature.SeqFeature(location, feature_dict['type'],
TypeError: SeqFeature.__init__() got an unexpected keyword argument 'strand'


An error occurred (see above) that has not been properly handled by Augur.
To report this, please open a new issue including the original command and the error above:
    <https://github.com/nextstrain/augur/issues/new/choose>

@victorlin victorlin merged commit bb84a44 into master Dec 22, 2023
66 checks passed
@victorlin victorlin deleted the victorlin/pin-biopython branch December 22, 2023 22:13
@victorlin victorlin mentioned this pull request Jan 31, 2024
1 task
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.

1 participant