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.

Sign up for GitHub

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

Jump to bottom

v0.5.1 #80

Merged
merged 2 commits into from
Feb 21, 2024
Merged

v0.5.1 #80

merged 2 commits into from
Feb 21, 2024

Conversation

nebfield
Copy link
Member

@nebfield nebfield commented Feb 21, 2024 •
edited
Loading

Changelog

  • Make aggregated_scores output consistent with ancestry_analysis output
  • Fix aggregate_scores not respecting --outdir
  • bump dependency versions with high security vulnerabilities

nebfield and others added 2 commits February 20, 2024 16:19
@nebfield
update vulnerable dependencies
d9c7389
@nebfield @smlmbrt
Update aggregation to match ancestry output (#79)
a4df14d 
* match ancestry aggregation output

* bump version

* fix column name (accession -> PGS)

* fix column name

* add aggregate tests

* fix not respecting outdir

* read new version of pgs

* drop onlySUM parameter

* Make sure it only reads SUM and provides the correct column names back

Signed-off-by: smlmbrt <sam.a.lambert@gmail.com>

* drop deprecated parameter

---------

Signed-off-by: smlmbrt <sam.a.lambert@gmail.com>
Co-authored-by: smlmbrt <sam.a.lambert@gmail.com>
@nebfield nebfield marked this pull request as ready for review February 21, 2024 12:37
@nebfield nebfield merged commit cf5d0d5 into main Feb 21, 2024
3 checks passed
@nebfield nebfield deleted the dev branch February 21, 2024 13:24
@nebfield nebfield mentioned this pull request Feb 21, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smlmbrt smlmbrt smlmbrt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nebfield @smlmbrt