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

Implement Preprocess DAG following recent architecture updates [Duplicate] #225

Closed
wants to merge 13 commits into from

Conversation

tskir
Copy link
Contributor

@tskir tskir commented Nov 5, 2023

Just repeating #216 which I accidentally merged into the wrong branch -__-

@codecov-commenter
Copy link

Codecov Report

Merging #225 (ca459e0) into main (2014e48) will increase coverage by 0.10%.
Report is 14 commits behind head on main.
The diff coverage is 86.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #225      +/-   ##
==========================================
+ Coverage   85.53%   85.64%   +0.10%     
==========================================
  Files          80       81       +1     
  Lines        1839     1853      +14     
==========================================
+ Hits         1573     1587      +14     
  Misses        266      266              
Files Coverage Δ
src/airflow/dags/common_airflow.py 100.00% <100.00%> (ø)
src/airflow/dags/dag_genetics_etl.py 100.00% <100.00%> (ø)
src/airflow/dags/dag_preprocess.py 100.00% <100.00%> (ø)
src/otg/datasource/finngen/__init__.py 100.00% <ø> (ø)
src/otg/datasource/finngen/summary_stats.py 100.00% <100.00%> (ø)
src/otg/finngen.py 65.38% <0.00%> (ø)

@tskir tskir closed this Nov 5, 2023
@tskir tskir deleted the tskir-3106-ingest-finngen branch November 5, 2023 10:23
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.

2 participants