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

Dev #23

Merged
merged 20 commits into from
Jan 15, 2024
Merged

Dev #23

merged 20 commits into from
Jan 15, 2024

Conversation

paulstretenowich
Copy link
Contributor

No description provided.

paulstretenowich and others added 17 commits October 19, 2023 15:47
@paulstretenowich
Adding digest_unanalyzed route + debug Operation being linked to Read…
588803a 
…set when ingesting transfer and GenPipes
@paulstretenowich
Debug missing arg
d262f87
@paulstretenowich
Using project rather than project_name or project_id in routes - test 1
d91dcfc
@paulstretenowich
Adding project converter/checker
8a982ee
@paulstretenowich
Debug pytest
83bc30c
@paulstretenowich
Adding if to avoid failure if project_name in json set to null
84bbdc0
@paulstretenowich
Small fix
85e6a2f
@paulstretenowich
Need to convert project_id from name_to_id from list of int into str
e6389d2
@paulstretenowich
Merge branch 'main' into dev
bc43e8a
@paulstretenowich
Adding a new attribute for experiment: nucleic_acid_type
d935efd
@paulstretenowich
Debug
dae290a
@paulstretenowich
Consolidating docstring + adding experiment_nucleic_acid_type to dige…
66e1944 
…st_unanalyzed output
@paulstretenowich
Code cleaning + docstring consolidation
620fdf4
@paulstretenowich
Changing error message
8674a8a
@paulstretenowich
Accepting metric without flag set
0f80cf8
@paulstretenowich
Allowing job_status being null if job hasn't been submitted
24319cb
@paulstretenowich
Skipping null job during ingesting genpipes as we don't want files no…
2375fdd 
…t generated
@poquirion
Copy link
Contributor

I will close this utill we decide to merge because it is eating up action credit for nothing right now.

poquirion
poquirion approved these changes Jan 15, 2024
View reviewed changes
Copy link
Contributor

@poquirion poquirion left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@poquirion poquirion merged commit a1d4448 into main Jan 15, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@poquirion poquirion poquirion 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
@paulstretenowich @poquirion